You're now a member of xooki, you should be able to commit your change. Xavier
On 10/29/07, Gilles Scokart <[EMAIL PROTECTED]> wrote: > > Ok, I can make the check-in in xooki codebase. My sourceforge userid is > gscokart. > > Gilles > > > 2007/10/27, Xavier Hanin <[EMAIL PROTECTED]>: > > > > On 10/27/07, Gilles Scokart <[EMAIL PROTECTED]> wrote: > > > > > > Thanks for your proposition #1, but I have unfortunately too few time > > > available to invest in the xooki project. > > > > > > For this particular case, the choice between #2 and #3 are depending > if > > > the > > > extension I made is usefull in the xooki code base or not. > > > > > > If you you want, I can have a look to see how to use the config.js > . But > > > if > > > you want to have this code in the xooki codebase (maybe with some > > > enhancements), then it is useless. > > > > > > I think your enhancement can be useful in xooki code base, so if it's > > allright for you we'll go with #2. But note that having write access to > > xooki svn doesn't engage you to do anything else than commit your change > > yourself. So if you already have a sourceforge.net user, I think it's > one > > of > > the best way to go. IMO #3 will take you more time. And if you still > > prefer > > #2, I won't be offended, and I'll push your changes back to xooki > myself, > > no > > problem. > > > > Xavier > > > > Gilles > > > > > > 2007/10/24, Xavier Hanin <[EMAIL PROTECTED]>: > > > > > > > > On 10/24/07, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: > > > > > > > > > > Author: gscokart > > > > > Date: Wed Oct 24 12:35:34 2007 > > > > > New Revision: 587977 > > > > > > > > > > URL: http://svn.apache.org/viewvc?rev=587977&view=rev > > > > > Log: > > > > > add support for include using [<URL>] > > > > > > > > > > Modified: > > > > > incubator/ivy/site/xooki/xooki.js > > > > > > > > > > > > As I think you know xooki is a separate project from Ivy, so > modifying > > > > xooki.js without modifying it in the original project is risky: we > may > > > > have > > > > problem with later upgrade. > > > > > > > > So I see several options: > > > > 1) I give you commit access to xooki, so that you can commit this > > change > > > > in > > > > xooki too > > > > 2) I push your change to xooki myself, and whenever one of us need > to > > > > improve xooki we discuss it here and I take care of maintaining the > > > > original > > > > xooki.js in sync > > > > 3) since your change is "just" a new input type, I think this can be > > > done > > > > externally to xooki.js using the config.js file. and thus do not > cause > > > > problem for later upgrade > > > > > > > > I'm personnally in favor of #1, then #3, then #2. > > > > > > > > WDYT? > > > > > > > > Xavier > > > > > > > > Modified: incubator/ivy/site/xooki/xooki.js > > > > > URL: > > > > > > > > > > > > > > > http://svn.apache.org/viewvc/incubator/ivy/site/xooki/xooki.js?rev=587977&r1=587976&r2=587977&view=diff > > > > > > > > > > > > > > > > > > > > ============================================================================== > > > > > --- incubator/ivy/site/xooki/xooki.js (original) > > > > > +++ incubator/ivy/site/xooki/xooki.js Wed Oct 24 12:35:34 2007 > > > > > @@ -781,6 +781,22 @@ > > > > > return '<br class="xooki-br"/>'; // the class > > is > > > > not > > > > > really necessary but allow to distinguish generated br from input > > one > > > > > } > > > > > }); > > > > > + }, > > > > > + > > > > > + includes: function (input) { > > > > > + //[<url>] replaced by the content of the url > > > > > + result = ""; > > > > > + lastStart = 0; > > > > > + nextPos = input.indexOf("[<" , lastStart); > > > > > + while( nextPos > 0 ) { > > > > > + result = result + input.slice > > > > (lastStart,nextPos); > > > > > + lastStart = nextPos; > > > > > + nextPos = input.indexOf(">]" , lastStart); > > > > > + result = result + xooki.url.loadURL(u( > > > > input.slice > > > > > (lastStart+2,nextPos))); > > > > > + lastStart = nextPos + 2; > > > > > + nextPos = input.indexOf("[<" , lastStart); > > > > > + } > > > > > + return result + input.slice(lastStart); > > > > > } > > > > > }, > > > > > > > > > > @@ -884,7 +900,7 @@ > > > > > xooki.c.initProperty("xookiInputFormat", ["xooki"]); > > > > > xooki.c.initProperty("allowEdit", !batchMode && > > > > xooki.pageURL.substr(0,5) > > > > > == "file:"); > > > > > > > > > > - xooki.input.format.define("xooki", ["code", "shortcuts", > "url", > > > > > "xookiLinks", "jira", "lineBreak"]); > > > > > + xooki.input.format.define("xooki", ["code", "shortcuts", > "url", > > > > > "xookiLinks", "jira", "lineBreak" , "includes"]); > > > > > > > > > > xooki.c.path = (typeof xooki.c.path != > > > > "undefined")?xooki.c.path:{}; > > > > > xooki.c.path.initProperty = initConfigProperty; > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > Xavier Hanin - Independent Java Consultant > > > > http://xhab.blogspot.com/ > > > > http://ant.apache.org/ivy/ > > > > http://www.xoocode.org/ > > > > > > > > > > > > > > > > -- > > > Gilles SCOKART > > > > > > > > > > > -- > > Xavier Hanin - Independent Java Consultant > > http://xhab.blogspot.com/ > > http://ant.apache.org/ivy/ > > http://www.xoocode.org/ > > > > > > -- > Gilles SCOKART > -- Xavier Hanin - Independent Java Consultant http://xhab.blogspot.com/ http://ant.apache.org/ivy/ http://www.xoocode.org/
