Re: [xwiki-devs] [Contrib] Create a repository for the Hierarchy Macro

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

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

features (which are cool). We could have it as a new extension until we implement all the features

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, vincent(a)massol.net wrote: > > > > > On 21 May 2014 at 18:08:33, Lyes Bandou (lyes.bandou(a)xwiki.com(mailto: > lyes.bandou(a)xwiki.com)) 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

> > load the data (using JSON) and the older macro display the documents > > hierarchy in a space , the new macro can uses documents list as

> > 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, vincent(a)massol.net > > > wrote: > > > > Do you really need a new extension? Couldn’t you improve the

> > > one then? > > > > > > > > Thanks > > > > -Vincent > > > > > > > > On 21 May 2014 at 17:35:52, Lyes Bandou (lyes.bandou(a)xwiki.com > (mailto: > > > lyes.bandou(a)xwiki.com)) wrote: > > > > > > > >> This is the new version of Hierarchy Macro ( > > > >>

> ) > > > 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

> 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 (

_______________________________________________ devs mailing list devs(a)xwiki.org http://lists.xwiki.org/mailman/listinfo/devs