Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(411)

Issue 49060: Template Library support for Shindig Java

Can't Edit
Can't Publish+Mail
Start Review
Created:
16 years, 9 months ago by levik
Modified:
11 years, 1 month ago
Reviewers:
opensocial.evan, shindig.remailer, awiner
Base URL:
http://svn.apache.org/repos/asf/incubator/shindig/trunk/
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+425 lines, -85 lines) Patch
M java/gadgets/src/main/java/org/apache/shindig/gadgets/GadgetException.java View 1 chunk +1 line, -0 lines 0 comments Download
M java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/TemplateRewriter.java View 8 chunks +71 lines, -17 lines 0 comments Download
A java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/AbstractTagRegistry.java View 1 chunk +32 lines, -0 lines 0 comments Download
A java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/CompositeTagRegistry.java View 1 chunk +41 lines, -0 lines 0 comments Download
A java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/DefaultTagRegistry.java View 1 chunk +52 lines, -0 lines 0 comments Download
M java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/TagRegistry.java View 1 chunk +6 lines, -58 lines 0 comments Download
A java/gadgets/src/main/java/org/apache/shindig/gadgets/templates/TemplateLibrary.java View 1 chunk +135 lines, -0 lines 0 comments Download
M java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/TemplateRewriterTest.java View 2 chunks +16 lines, -4 lines 0 comments Download
M java/gadgets/src/test/java/org/apache/shindig/gadgets/templates/DefaultTemplateProcessorTest.java View 1 chunk +1 line, -1 line 0 comments Download
M java/gadgets/src/test/java/org/apache/shindig/gadgets/templates/RenderTagHandlerTest.java View 4 chunks +8 lines, -4 lines 0 comments Download
M java/gadgets/src/test/java/org/apache/shindig/gadgets/templates/TemplateBasedTagHandlerTest.java View 1 chunk +2 lines, -1 line 0 comments Download
A java/gadgets/src/test/java/org/apache/shindig/gadgets/templates/TemplateLibraryTest.java View 1 chunk +60 lines, -0 lines 0 comments Download

Messages

Total messages: 1
awiner
16 years, 9 months ago (2009-04-24 02:18:32 UTC) #1
Looks good.  I'll clean up some of the details - e.g., don't use
getElementsByTagName(), which descends into children, and add some caching of
parse results and support for <JavaScript> and <Style> elements.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b