Re: Send HTTP status codes with GadgetException throws (issue207060)

2010-02-11 Thread lindner
logic seems fine to me. http://codereview.appspot.com/207060/show

Re: Socialsite mavenized status update. please help!

2010-02-11 Thread Viggo Navarsete
Thanks for the reply Paul. Based on what's in the mavenized-socialsite project, could you be more concrete about what needs to be changed/fixed? Regards, Viggo On Thu, Feb 11, 2010 at 8:56 AM, Paul Lindner lind...@inuus.com wrote: It's possible that the containing page is not including

Re: how to disable javascript cache for default container?

2010-02-11 Thread Henry Saputra
HI Robert, I believe you can add optional request parameters nocache=1 to the gadget iframe url to force gadget rendering servlet to re-render the gadget. - Henry On Wed, Feb 10, 2010 at 6:39 PM, Robert Peterson juzzam.shin...@gmail.comwrote: We've only been using Shindig for a couple days -

Re: Send HTTP status codes with GadgetException throws (issue207060)

2010-02-11 Thread John Hjelmstad
Thanks for looking, Paul! I've committed this CL. Happy to accept refinements over time. On Thu, Feb 11, 2010 at 12:18 AM, lind...@inuus.com wrote: logic seems fine to me. http://codereview.appspot.com/207060/show

Re: Continued improvement to exception handling code (issue207059)

2010-02-11 Thread John Hjelmstad
On Thu, Feb 11, 2010 at 2:57 AM, m...@google.com wrote: http://codereview.appspot.com/207059/diff/1/14 File java/gadgets/src/main/java/org/apache/shindig/gadgets/AbstractSpecFactory.java (right): http://codereview.appspot.com/207059/diff/1/14#newcode134

html_sanitize is not defined

2010-02-11 Thread Pieter Maes -- Maescool
Hi, When testing the beta5 release (from the tags in svn) i stumbled upon this error When grepping where it was used features/src/main/javascript/features/tabs/tabs.js: tab.td_.innerHTML = html_sanitize(tabName); features/src/main/javascript/features/minimessage/minimessage.js: td.innerHTML

Re: html_sanitize is not defined

2010-02-11 Thread ๏̯͡๏ Jasvir Nagra
It ought to be fetched by maven and stashed in your maven cache ~/.m2/repository/caja/caja/r3950/caja-r3950.jar where 3950 is the version of caja its pulling. I am not sure how the php version unpacks this file to get at html-sanitizer.js On Thu, Feb 11, 2010 at 4:20 PM, Pieter Maes -- Maescool

Re: html_sanitize is not defined

2010-02-11 Thread Paul Lindner
Since PHP doesn't have the caja jar in the classpath the res:// URL is converted into an http URL // Make html-santitization work see SHINDIG-346 if ($content == 'res://com/google/caja/plugin/html-sanitizer-minified.js') { $content= '

Re: html_sanitize is not defined

2010-02-11 Thread ๏̯͡๏ Jasvir Nagra
As long as you're using the latest caja, this scheme should work fine. Of course if you're using a different version of caja, there'll be version skew issues if you're hard linking to the latest html-sanitizer. That said, html-sanitizer api and code has been pretty stable for the last year.