> On 2011-10-25 07:02:51, Santhosh Srinivasan wrote: > > /trunk/core/src/test/java/org/apache/oozie/command/wf/TestActionStartXCommand.java, > > line 200 > > <https://reviews.apache.org/r/2427/diff/5/?file=52971#file52971line200> > > > > Is this a typo?
it is not. it is valid syntax of javadoc. - Angelo K. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2427/#review2815 ----------------------------------------------------------- On 2011-10-25 01:43:42, Angelo K. Huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2427/ > ----------------------------------------------------------- > > (Updated 2011-10-25 01:43:42) > > > Review request for oozie. > > > Summary > ------- > > try { > String valueElem = "<value>"+value+"</value>"; > XmlUtils.parseXml(valueElem); > } > catch (JDOMException ex) { > // It should not happen, so escape the characters for xml > value = XmlUtils.escapeCharsForXML(value); > } > > With above check, for element CDATA can be avoided for escaping, EX: > > For these two elements, first one has to convert because of '&', however > second one can be avoided. > > <property> > <name>test.ampsign</name> > > <value>http://app1.soln-stage.nova.cp.vip.ne1.yahoo.com/nova-webservices?urlSigner=signUrl&namespace=nova.proxy</value> > </property> > <property> > <name>test.cdata</name> > > <value><![CDATA[?redirect=http%3A%2F%2Fapp1.soln-stage.nova.cp.vip.ne1.yahoo.com%3A4080%2Fnova-webservices%2Fv1%2FurlSigner%2FsignUrl&namespace=nova.proxy&keyDBHash=Vsy6n_C7K6NG0z4R2eBlKg--]]></value> > </property> > > > *** & has to convert to & *** > *** <![CDATA[]] does not need to convert. *** > > <property> > <name>test.ampsign</name> > > <value>http://app1.soln-stage.nova.cp.vip.ne1.yahoo.com/nova-webservices?urlSigner=signUrl&namespace=nova.proxy</value> > </property> > <property> > <name>test.cdata</name> > > <value><![CDATA[?redirect=http%3A%2F%2Fapp1.soln-stage.nova.cp.vip.ne1.yahoo.com%3A4080%2Fnova-webservices%2Fv1%2FurlSigner%2FsignUrl&namespace=nova.proxy&keyDBHash=Vsy6n_C7K6NG0z4R2eBlKg--]]></value> > </property> > > > This addresses bug OOZIE-580. > https://issues.apache.org/jira/browse/OOZIE-580 > > > Diffs > ----- > > /trunk/core/src/main/java/org/apache/oozie/DagELFunctions.java 1185461 > > /trunk/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java > 1185461 > > /trunk/core/src/test/java/org/apache/oozie/command/wf/TestActionStartXCommand.java > 1185461 > /trunk/release-log.txt 1185461 > > Diff: https://reviews.apache.org/r/2427/diff > > > Testing > ------- > > > Thanks, > > Angelo K. > >
