Hello Devs, As far as I know it's not a refactoring of the old macro because some parameters / parameters behaviour was not kept 100%. It's missing some features, basically. Adding some with the cost of removing others.
So if we consider it a new version of the old one, installing an "upgrade" will break the behaviour. You'd tell me we could wait to release the new version it only when ready, but then people could not take advantage of its features (which are cool). We could have it as a new extension until we implement all the features of the old one, and then we mark it as implementing the same feature so that the old one could be upgraded to the new one. Anca On Wed, May 21, 2014 at 7:11 PM, [email protected] <[email protected]>wrote: > > > > > On 21 May 2014 at 18:08:33, Lyes Bandou ([email protected](mailto: > [email protected])) wrote: > > > Code of the two macros is not the same, > > This doesn’t matter much if the new code is considered a refactoring of > the old code. > > > The new macro has the same name and > > keeps most paramettres, but the behavior is not the same it uses Ajax to > > load the data (using JSON) and the older macro display the documents > > hierarchy in a space , the new macro can uses documents list as documents > > roots of the tree (plus another paramettres) > > > > > > I suggest "Dynamic Tree Macro" as the new name. > > The real question is: do we need the 2 macros at once? > > I guess we need to ask the question to the author of the first macro: > Anca, Paul, Mircea. > > Because from a user point of view, it’s a nightmare to have several > extensions doing the same thing, the user never knows which one to use. > > Anca/Paul/Mircea, do you have an opinion? > > Thanks > -Vincent > > > On Wed, May 21, 2014 at 4:40 PM, Thomas Mortagne > > wrote: > > > > > Same question, why is a new repository needed exactly ? If it's THE > > > new version then it's the same extension and it should be the same > > > repository, even if you rewrite half of it in a 2.0 version. > > > > > > On Wed, May 21, 2014 at 5:37 PM, [email protected] > > > wrote: > > > > Do you really need a new extension? Couldn’t you improve the existing > > > one then? > > > > > > > > Thanks > > > > -Vincent > > > > > > > > On 21 May 2014 at 17:35:52, Lyes Bandou ([email protected] > (mailto: > > > [email protected])) wrote: > > > > > > > >> This is the new version of Hierarchy Macro ( > > > >> http://extensions.xwiki.org/xwiki/bin/view/Extension/HierarchyMacro > ) > > > with > > > >> the addition of to many parameters > > > >> > > > >> > > > >> On Wed, May 21, 2014 at 4:26 PM, Thomas Mortagne > > > >> wrote: > > > >> > > > >> > Are you talking about > > > >> > > http://extensions.xwiki.org/xwiki/bin/view/Extension/HierarchyMacro ? > > > >> > > > > >> > If not you might want to find another name since this one already > have > > > >> > https://github.com/xwiki-contrib/macro-hierarchy. > > > >> > > > > >> > On Wed, May 21, 2014 at 5:23 PM, Lyes Bandou > > > >> > wrote: > > > >> > > Hello devs, > > > >> > > > > > >> > > I would like a repository for the Hierarchy Macro on > > > >> > > https://github.com/xwiki-contrib. > > > >> > > > > > >> > > Hierarchy Macro use jQuery and jsTree (http://www.jstree.com) > to > > > display > > > >> > > dynamique tree view of wiki documents based on parent/child > > > relationships > > > >> > > > > > >> > > Thank you in advance. > > > >> > > > > > >> > > > > > >> > > Lyes Bandou > > > > > > > > _______________________________________________ > > > > devs mailing list > > > > [email protected] > > > > http://lists.xwiki.org/mailman/listinfo/devs > > > > > > > > > > > > -- > > > Thomas Mortagne > > > _______________________________________________ > > > devs mailing list > > > [email protected] > > > http://lists.xwiki.org/mailman/listinfo/devs > > > > > _______________________________________________ > > devs mailing list > > [email protected] > > http://lists.xwiki.org/mailman/listinfo/devs > _______________________________________________ > devs mailing list > [email protected] > http://lists.xwiki.org/mailman/listinfo/devs > _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
