Marc Portier wrote:


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




Reply via email to