In its current state the refactoring module need to be put in the
minimal package. Ideally there should be several differents packages,
one with the minimum and some extension point and then various
extensions providing hooks for renaming pages, etc.
I would say lets put it in the minimal package and create an
improvement issue for the separation, it's not a blocker.
On Mon, Aug 31, 2015 at 5:54 PM, vincent(a)massol.net <vincent(a)massol.net> wrote:
Hi,
Right now almost all func tests fail to delete pages when they do
getUtil().deletePage(…). They fail with:
Caused by: org.xwiki.component.manager.ComponentLookupException: Can't find
descriptor for the component [role = [interface org.xwiki.job.Job] hint = [delete]]
The reason is that since XWIKI-12268 the Delete action cannot work if the Refactoring
module isn't bundled, which makes the Refactoring module a core module. And this
module is not part of the core modules ATM. I’ve added it to the PackageMojo for now but
we need to decide what to do.
Personally I’ve always viewed Refactoring as optional and advanced features.
I don’t consider basic delete, basic copy and basic rename/move as refactoring options.
However there are some parts of them that are refactorings. For example when Renaming,
also check for wiki pages having the renamed page as parent and updating them or updating
wiki links in pages.
So we need to decide if:
* We consider the Refactoring module as core.
* Introduce a new module and move some of the core actions such as delete, move, etc to
it and keep the refactoring module for non-core refactoring
* Split the refactoring into basic refactorings and advanced ones. Have basic delete,
copy, rename + advanced options for them separately
* Others?
WDYT?
Still hesitating on my side.
Thanks
-Vincent
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
--
Thomas Mortagne