comparison mercurial/transaction.py @ 49994:3128018e878b

transaction: run abort callback in all cases

Previously, these possibly important callback were "forgotten" when running a
quick rollback.

This is now fixed, as the tests shown.

49993:2f348babe30d

        self._file.close()
        self._backupsfile.close()

        quick = self._can_quick_abort(entries)
        try:
            if quick:
                self._do_quick_abort(entries)
            else:
                self._do_full_abort(entries)
        finally:

        if self._journal:
            self._opener.unlink(self._journal)

    def _do_full_abort(self, entries):
        """(Noisily) rollback all the change introduced by the transaction"""
        self._report(_(b"transaction abort!\n"))
        try:
            for cat in sorted(self._abortcallback):
                self._abortcallback[cat](self)
            # Prevent double usage and help clear cycles.
            self._abortcallback = None
            _playback(
                self._journal,
                self._report,
                self._opener,
                self._vfsmap,

49994:3128018e878b

        self._file.close()
        self._backupsfile.close()

        quick = self._can_quick_abort(entries)
        try:
            if not quick:
                self._report(_(b"transaction abort!\n"))
            for cat in sorted(self._abortcallback):
                self._abortcallback[cat](self)
            # Prevent double usage and help clear cycles.
            self._abortcallback = None
            if quick:
                self._do_quick_abort(entries)
            else:
                self._do_full_abort(entries)
        finally:

        if self._journal:
            self._opener.unlink(self._journal)

    def _do_full_abort(self, entries):
        """(Noisily) rollback all the change introduced by the transaction"""
        try:
            _playback(
                self._journal,
                self._report,
                self._opener,
                self._vfsmap,