----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22363/#review46020 -----------------------------------------------------------
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/render/RenderingGadgetRewriter.java <https://reviews.apache.org/r/22363/#comment81182> Why not combine these two if statements? It would simplify the code a bit. - Ryan Baxter On June 16, 2014, 3:07 a.m., Jiaqing Guo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22363/ > ----------------------------------------------------------- > > (Updated June 16, 2014, 3:07 a.m.) > > > Review request for shindig, Dan Dumont, Ryan Baxter, Rich Thompson, and > Stanton Sievers. > > > Bugs: SHINDIG-1978 > https://issues.apache.org/jira/browse/SHINDIG-1978 > > > Repository: shindig > > > Description > ------- > > If gadget spec contains a "ModulePrefs" tag with "title" attribute like below: > <ModulePrefs title="this is title" description="Hello World Embedded > Experience gadget" height="400" width="500"> > Then expect the loaded gadget should contain proper "title" element like > below: > <html> > <head> > <title>this is title</title> > ...... > But the actual result is the load gadget doesn't contain "title" element. > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/render/RenderingGadgetRewriter.java > 1436455 > > Diff: https://reviews.apache.org/r/22363/diff/ > > > Testing > ------- > > > Thanks, > > Jiaqing Guo > >