Re: Review Request: Gadget Administration - Securing Features and RPC

2011-09-30 Thread Henry Saputra
> On 2011-09-30 22:37:19, Henry Saputra wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/admin/GadgetAdminModule.java, > > line 54 > > > > > > Why do we inject

Re: Review Request: Gadget Administration - Securing Features and RPC

2011-09-30 Thread Ryan Baxter
> On 2011-09-30 22:37:19, Henry Saputra wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/admin/GadgetAdminModule.java, > > line 54 > > > > > > Why do we inject

Re: Review Request: Gadget Administration - Securing Features and RPC

2011-09-30 Thread Henry Saputra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1666/#review2240 --- http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/j

Re: Switching views

2011-09-30 Thread Ryan J Baxter
No problem it happens to me all the time :) -Ryan Email: rjbax...@us.ibm.com Phone: 978-899-3041 developerWorks Profile From: daviesd To: , Date: 09/30/2011 04:17 PM Subject:Re: Switching views I apologize. It's Friday and my brain has already left the office. This IS w

Re: Review Request: Locked domain cleanup and shared-domain-locking feature

2011-09-30 Thread Dan Dumont
> On 2011-09-30 14:14:44, Jesse Ciancetta wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/HashLockedDomainService.java, > > line 164 > > > > > > It looks lik

Re: Switching views

2011-09-30 Thread daviesd
I apologize. It's Friday and my brain has already left the office. This IS working for me. In fact Ryan, I do not have to close the gadget site. It navigates to the new view properly. Thanks for the help. If anything it got me to use the osapi syntax properly, which I was not doing. doug O

Re: Switching views

2011-09-30 Thread daviesd
Ryan, We had created our own shindig snapshot back on 9/23 because we were hitting a deadline and didn't want things to change. So I thought that might be the problem. But I just synched up with the latest shindig and it's still not working for me. Our old example that use to work that creates

RE: Review Request: Common container currently doesnt include the siteId (moduleId) in any of it's security token processing/handling

2011-09-30 Thread Ryan J Baxter
Couldn't I reuse a site though? So if I have one gadget rendered in a site and then I close it and reuse that site to render a new gadget wouldn't both gadgets then have the same module id? Or maybe that doesn't matter since you closed the original gadget -Ryan Email: rjbax...@us.ibm.co

Re: Review Request: Locked domain cleanup and shared-domain-locking feature

2011-09-30 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2025/ --- (Updated 2011-09-30 19:56:46.329747) Review request for shindig, Paul Lindner, j

Re: Switching views

2011-09-30 Thread Ryan J Baxter
Yeah Dan I just tried this internally, with a sync we did yesterday from Shindig, it seems to work fine in our demo app. Doug, have you tried closing the gadget site before navigating to the new view? I think this is how we do it. -Ryan Email: rjbax...@us.ibm.com Phone: 978-899-3041 developer

Re: Switching views

2011-09-30 Thread Dan Dumont
I'll try to look into this soon. Under a deadline to get our patches reviewed and delivered. I'm pretty sure we have this working fine internally, I'll see what I can set up and check. From: daviesd To: , Date: 09/30/2011 03:40 PM Subject:Re: Switching views Dan, I wen

Re: Switching views

2011-09-30 Thread daviesd
Dan, I went back to an example we've used for a while to demonstrate changing views and it no longer works either. I wonder if something changed recently that broke this? doug On 9/30/11 3:12 PM, "daviesd" wrote: > My bad... I wasn't passing the second parameter correctly (the gadget url).

Review Request: Add a feature for firebug-lite

2011-09-30 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2134/ --- Review request for shindig, Bastian Hofmann and johnfargo. Summary --- Add

Re: Review Request: Gadget Administration - Securing Features and RPC

2011-09-30 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1666/ --- (Updated 2011-09-30 19:21:13.986376) Review request for shindig, Paul Lindner, H

Re: Switching views

2011-09-30 Thread daviesd
My bad... I wasn't passing the second parameter correctly (the gadget url). I am doing it correctly now. Firebug errors go away, but the view still does not change. On 9/30/11 3:04 PM, "daviesd" wrote: > I don't have a public accessible site, sorry. > > Here's what I have now (not sure if I sh

Re: Switching views

2011-09-30 Thread daviesd
I don't have a public accessible site, sorry. Here's what I have now (not sure if I should be tweaking view or render params). var viewParams = {}; viewParams[osapi.container.ViewParam.VIEW] = 'home'; var renderParams = {}; renderParams[osapi.container.RenderParam.VIEW] = 'home';

Re: Switching views

2011-09-30 Thread Dan Dumont
Hmmm.. Though you can, I wouldn't start the render chain from the gadget site. There is important lifecycle things that happen in container. navigateGadget So you verified that the key being passed in was cased properly (as the constant defined in RenderParam)? Do you have a public url that I c

Re: Switching views

2011-09-30 Thread daviesd
Thanks Dan. Ya, even with those suggestions I'm not getting it to switch. I also tried. gadgetSite.render(gadgetSite.getActiveGadgetHolder().getGadgetInfo(), {}, renderParams); Just wondering what the intention is here. For example we have a 'preferences' view and my gadget has a menu item that

Re: Switching views

2011-09-30 Thread Dan Dumont
constants* From: Dan Dumont/Westford/IBM@Lotus To: dev@shindig.apache.org, Date: 09/30/2011 02:39 PM Subject:Re: Switching views gadgetSite.container_ is a private member, I'd avoid using it. I think the intention is that you would save your common container instance as a gl

Re: Switching views

2011-09-30 Thread Dan Dumont
gadgetSite.container_ is a private member, I'd avoid using it. I think the intention is that you would save your common container instance as a global or easily accessible reference. Also, params are case sensitive. It's safest, though not always necessary to use the constance defined in the re

Re: Review Request: OAuth2.0 client implementation in Apache Shindig.

2011-09-30 Thread li xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1947/ --- (Updated 2011-09-30 18:38:20.349435) Review request for shindig. Changes -

Switching views

2011-09-30 Thread daviesd
What is the common container way of switching the view of an already rendered gadget? I tried this var renderParams = { view: 'canvas' }; gadgetSite.container_.navigateGadget(gadgetSite, gadget, {}, renderParams); But it¹s not doing anything. doug

Re: Review Request: Locked domain cleanup and shared-domain-locking feature

2011-09-30 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2025/#review2219 --- http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/j

Ignoring JSON fields

2011-09-30 Thread chico charlesworth
Hi, Is there any way to configure Shindig to ignore certain class properties when converting to JSON, something like JsonIgnoreProperties[1] when using the Jackson JSON processor? [1] http://jackson.codehaus.org/1.4.2/javadoc/org/codehaus/jackson/annotate/JsonIgnoreProperties.html Cheers, Chico

RE: Review Request: Common container currently doesnt include the siteId (moduleId) in any of it's security token processing/handling

2011-09-30 Thread Ciancetta, Jesse E.
>-Original Message- >From: Ryan J Baxter [mailto:rjbax...@us.ibm.com] >Sent: Thursday, September 29, 2011 1:54 PM >To: Ciancetta, Jesse E. >Cc: dev@shindig.apache.org; Stanton Sievers >Subject: RE: Review Request: Common container currently doesnt include the >siteId (moduleId) in any of it

Re: Review Request: Locked domain cleanup and shared-domain-locking feature

2011-09-30 Thread Jesse Ciancetta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2025/#review2216 --- Overall the patch looks pretty good to me -- it's nice to see the clea

Re: Inline rendering and 3.0 Release

2011-09-30 Thread Rob Keane
Nevermind looks like I had the 2.0 and 3.0 patches mixed up. Sorry about that! On Sep 29, 2011, at 10:03 AM, Michael Beaver wrote: > It's applicable to the current trunk, and you should be able to apply it to > fairly recent extracts of trunk as well. > > > > Michael Beaver > IBM Lotus > 9

Re: Inline rendering and 3.0 Release

2011-09-30 Thread Rob Keane
Has anyone else been able to successfully apply this patch to the trunk and build? I had some errors applying and the patch seems to reference some methods that don't exist e.g. ConcatUriManaget.ConcatData.getUri which doesn't seem to be part of the patch and is not in trunk. On Sep 29, 2011,

Re: Including Firebug Lite As A Feature In Shindig-Extras

2011-09-30 Thread Bastian Hofmann
+1 On Fri, Sep 30, 2011 at 3:49 AM, Ryan J Baxter wrote: > Already have code mocked up, its actually very simple.  Got to love the > easy features that give you such a big payoff! > > -Ryan > > Email: rjbax...@us.ibm.com > Phone: 978-899-3041 > developerWorks Profile > > > > From:   John Hjelmsta