Last week, I greatly simplified our 'revert' code. However, in the process, I changed the notifications from 'revert' too:
The old code would send notifications for all modified nodes (including tree modifications), with a single exception: it would send a notification only for the root in case of non-replaced-added/copied/moved nodes. The change that I made is to extend 'notify-on-root-only' to added (non-copied/moved) nodes which are also replacements. However, talking to Bert, he said he'd rather get more notifications than fewer and decide himself whether he wants to show them in his GUI. This made me think we may want to distinguish two or three notification types for a path: * Content/props-restored paths * Removed-from-version-control paths (invoked for non-replaced added/copied/moved paths) * Restored paths (invoked for deleted/replaced paths) Optionally, it would be possible to use different notifications for paths which are not op_roots, but which *are* part of a tree modification - let's call those 'derived' paths. Ofcourse, we'd then need to decide which notifications our client would show. Thoughts? Comments? Bye, Erik.
