Sylvain Wallez wrote:
Marc Portier wrote:
Yo people,
after the recent introduction of the WoodyPipelineConfig object, I am tempted to change the line 125 in woody2.js
see: http://cvs.apache.org/viewcvs.cgi/cocoon-2.1/src/blocks/woody/java/org/apache/cocoon/woody/flow/javascript/woody2.js?rev=1.3&view=auto
for: bizData["woody-form"] = this.form;
towards:
bizData[Packages.org.apache.cocoon.woody.transformation.WoodyPipelineConfig.WOODY_FORM] = this.form;
this is a natural reflex from pure Java land, but I'm left to doubth now it this kind of more tight integration is desirable into the js-area?
It *is* desirable, as the JS and Java code will use this string to communicate. So it's better to have it defined in a single location.
So +1!
thx for confirming... change underway...
PS Many tools (read: IDEA) will search in text files for your Java identifiers when renaming/deleting.
Vadim