You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by "Erel Segal (JIRA)" <ji...@apache.org> on 2008/07/02 18:25:45 UTC

[jira] Created: (SHINDIG-420) web_prefix is not used in container.js

web_prefix is not used in container.js
--------------------------------------

                 Key: SHINDIG-420
                 URL: https://issues.apache.org/jira/browse/SHINDIG-420
             Project: Shindig
          Issue Type: Bug
          Components: Features (Javascript), OpenSocial  feature (Javascript)
         Environment: all
            Reporter: Erel Segal


In shindig/config/container.js, the web_prefix setting is not used.

For example, in "opensocial-0.8", there is a line:
    "path" : "/social/data"

It should include the web_prefix, otherwise the samples don't work.

I don't know how to add it, since it is defined in PHP!

If this requires the user to change something in a JS file, I think the documentation here:
   http://incubator.apache.org/shindig/#php
should be updated as well.


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Re: [jira] Resolved: (SHINDIG-420) web_prefix is not used in container.js

Posted by Erel Segal <er...@gmail.com>.
Please see the small documentation patch I attached here:
https://issues.apache.org/jira/secure/attachment/12386050/README.patch

Is this correct?

2008/7/3, Erel Segal <er...@gmail.com>:
>
> OK, but still we should have
>  * a javascript configuration file with a "web_prefix" setting, so that we
> won't have to change it in too many places inside the code;
>  * change the documentation, to alert users that they should change both
> web_prefix settings at the same time, if they decide to change it.
>
>
>
> 2008/7/2, Chris Chabot (JIRA) <ji...@apache.org>:
>>
>>
>>      [
>> https://issues.apache.org/jira/browse/SHINDIG-420?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel]
>>
>> Chris Chabot resolved SHINDIG-420.
>> ----------------------------------
>>
>>     Resolution: Won't Fix
>>       Assignee: Chris Chabot
>>
>> Hi Erel, I'm afraid this is not a bug.
>>
>> Shindig is currently implemented in 2 languages, PHP and Java. And while
>> the gadget/social server code is unique for each side, they share as much as
>> possible in terms of javascript related code.
>>
>> That means anything in features/*, javascript/* and config/* is shared by
>> both versions.
>>
>> Hence that file is not 'PHP' related, so you can't use any PHP variables
>> there :) When you change the web_prefix, you'll have to update the
>> config/container.js file too i'm afraid.
>>
>> > web_prefix is not used in container.js
>> > --------------------------------------
>> >
>> >                 Key: SHINDIG-420
>> >                 URL: https://issues.apache.org/jira/browse/SHINDIG-420
>> >             Project: Shindig
>> >          Issue Type: Bug
>> >          Components: Features (Javascript), OpenSocial  feature
>> (Javascript)
>> >         Environment: all
>> >            Reporter: Erel Segal
>> >            Assignee: Chris Chabot
>> >   Original Estimate: 1h
>> >  Remaining Estimate: 1h
>> >
>> > In shindig/config/container.js, the web_prefix setting is not used.
>> > For example, in "opensocial-0.8", there is a line:
>> >     "path" : "/social/data"
>> > It should include the web_prefix, otherwise the samples don't work.
>> > I don't know how to add it, since it is defined in PHP!
>> > If this requires the user to change something in a JS file, I think the
>> documentation here:
>> >    http://incubator.apache.org/shindig/#php
>> > should be updated as well.
>>
>> --
>> This message is automatically generated by JIRA.
>> -
>> You can reply to this email to add a comment to the issue online.
>>
>>
>

Re: [jira] Resolved: (SHINDIG-420) web_prefix is not used in container.js

Posted by Erel Segal <er...@gmail.com>.
OK, but still we should have
 * a javascript configuration file with a "web_prefix" setting, so that we
won't have to change it in too many places inside the code;
 * change the documentation, to alert users that they should change both
web_prefix settings at the same time, if they decide to change it.



2008/7/2, Chris Chabot (JIRA) <ji...@apache.org>:
>
>
>      [
> https://issues.apache.org/jira/browse/SHINDIG-420?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel]
>
> Chris Chabot resolved SHINDIG-420.
> ----------------------------------
>
>     Resolution: Won't Fix
>       Assignee: Chris Chabot
>
> Hi Erel, I'm afraid this is not a bug.
>
> Shindig is currently implemented in 2 languages, PHP and Java. And while
> the gadget/social server code is unique for each side, they share as much as
> possible in terms of javascript related code.
>
> That means anything in features/*, javascript/* and config/* is shared by
> both versions.
>
> Hence that file is not 'PHP' related, so you can't use any PHP variables
> there :) When you change the web_prefix, you'll have to update the
> config/container.js file too i'm afraid.
>
> > web_prefix is not used in container.js
> > --------------------------------------
> >
> >                 Key: SHINDIG-420
> >                 URL: https://issues.apache.org/jira/browse/SHINDIG-420
> >             Project: Shindig
> >          Issue Type: Bug
> >          Components: Features (Javascript), OpenSocial  feature
> (Javascript)
> >         Environment: all
> >            Reporter: Erel Segal
> >            Assignee: Chris Chabot
> >   Original Estimate: 1h
> >  Remaining Estimate: 1h
> >
> > In shindig/config/container.js, the web_prefix setting is not used.
> > For example, in "opensocial-0.8", there is a line:
> >     "path" : "/social/data"
> > It should include the web_prefix, otherwise the samples don't work.
> > I don't know how to add it, since it is defined in PHP!
> > If this requires the user to change something in a JS file, I think the
> documentation here:
> >    http://incubator.apache.org/shindig/#php
> > should be updated as well.
>
> --
> This message is automatically generated by JIRA.
> -
> You can reply to this email to add a comment to the issue online.
>
>

[jira] Resolved: (SHINDIG-420) web_prefix is not used in container.js

Posted by "Cassie Doll (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-420?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Cassie Doll resolved SHINDIG-420.
---------------------------------

    Resolution: Fixed
      Assignee: Cassie Doll  (was: Chris Chabot)

> web_prefix is not used in container.js
> --------------------------------------
>
>                 Key: SHINDIG-420
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-420
>             Project: Shindig
>          Issue Type: Bug
>          Components: Features (Javascript), OpenSocial  feature (Javascript)
>         Environment: all
>            Reporter: Erel Segal
>            Assignee: Cassie Doll
>         Attachments: README.patch
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In shindig/config/container.js, the web_prefix setting is not used.
> For example, in "opensocial-0.8", there is a line:
>     "path" : "/social/data"
> It should include the web_prefix, otherwise the samples don't work.
> I don't know how to add it, since it is defined in PHP!
> If this requires the user to change something in a JS file, I think the documentation here:
>    http://incubator.apache.org/shindig/#php
> should be updated as well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SHINDIG-420) web_prefix is not used in container.js

Posted by "Erel Segal (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-420?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Erel Segal updated SHINDIG-420:
-------------------------------

    Attachment: README.patch

A small addition to the documentation

> web_prefix is not used in container.js
> --------------------------------------
>
>                 Key: SHINDIG-420
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-420
>             Project: Shindig
>          Issue Type: Bug
>          Components: Features (Javascript), OpenSocial  feature (Javascript)
>         Environment: all
>            Reporter: Erel Segal
>            Assignee: Chris Chabot
>         Attachments: README.patch
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In shindig/config/container.js, the web_prefix setting is not used.
> For example, in "opensocial-0.8", there is a line:
>     "path" : "/social/data"
> It should include the web_prefix, otherwise the samples don't work.
> I don't know how to add it, since it is defined in PHP!
> If this requires the user to change something in a JS file, I think the documentation here:
>    http://incubator.apache.org/shindig/#php
> should be updated as well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SHINDIG-420) web_prefix is not used in container.js

Posted by "Erel Segal (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-420?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Erel Segal updated SHINDIG-420:
-------------------------------

    Attachment: README.patch

just fix the path

> web_prefix is not used in container.js
> --------------------------------------
>
>                 Key: SHINDIG-420
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-420
>             Project: Shindig
>          Issue Type: Bug
>          Components: Features (Javascript), OpenSocial  feature (Javascript)
>         Environment: all
>            Reporter: Erel Segal
>            Assignee: Chris Chabot
>         Attachments: README.patch
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In shindig/config/container.js, the web_prefix setting is not used.
> For example, in "opensocial-0.8", there is a line:
>     "path" : "/social/data"
> It should include the web_prefix, otherwise the samples don't work.
> I don't know how to add it, since it is defined in PHP!
> If this requires the user to change something in a JS file, I think the documentation here:
>    http://incubator.apache.org/shindig/#php
> should be updated as well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Reopened: (SHINDIG-420) web_prefix is not used in container.js

Posted by "Erel Segal (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-420?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Erel Segal reopened SHINDIG-420:
--------------------------------


OK, but still we should have
 * a Javascript configuration file with a "web_prefix" setting, so that we won't have to change it in too many places inside the code;
 * change the documentation, to alert users that they should change both web_prefix settings at the same time, if they decide to change it.

> web_prefix is not used in container.js
> --------------------------------------
>
>                 Key: SHINDIG-420
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-420
>             Project: Shindig
>          Issue Type: Bug
>          Components: Features (Javascript), OpenSocial  feature (Javascript)
>         Environment: all
>            Reporter: Erel Segal
>            Assignee: Chris Chabot
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In shindig/config/container.js, the web_prefix setting is not used.
> For example, in "opensocial-0.8", there is a line:
>     "path" : "/social/data"
> It should include the web_prefix, otherwise the samples don't work.
> I don't know how to add it, since it is defined in PHP!
> If this requires the user to change something in a JS file, I think the documentation here:
>    http://incubator.apache.org/shindig/#php
> should be updated as well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SHINDIG-420) web_prefix is not used in container.js

Posted by "Erel Segal (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-420?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Erel Segal updated SHINDIG-420:
-------------------------------

    Attachment:     (was: README.patch)

> web_prefix is not used in container.js
> --------------------------------------
>
>                 Key: SHINDIG-420
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-420
>             Project: Shindig
>          Issue Type: Bug
>          Components: Features (Javascript), OpenSocial  feature (Javascript)
>         Environment: all
>            Reporter: Erel Segal
>            Assignee: Chris Chabot
>         Attachments: README.patch
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In shindig/config/container.js, the web_prefix setting is not used.
> For example, in "opensocial-0.8", there is a line:
>     "path" : "/social/data"
> It should include the web_prefix, otherwise the samples don't work.
> I don't know how to add it, since it is defined in PHP!
> If this requires the user to change something in a JS file, I think the documentation here:
>    http://incubator.apache.org/shindig/#php
> should be updated as well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (SHINDIG-420) web_prefix is not used in container.js

Posted by "Chris Chabot (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-420?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Chris Chabot resolved SHINDIG-420.
----------------------------------

    Resolution: Won't Fix
      Assignee: Chris Chabot

Hi Erel, I'm afraid this is not a bug.

Shindig is currently implemented in 2 languages, PHP and Java. And while the gadget/social server code is unique for each side, they share as much as possible in terms of javascript related code.

That means anything in features/*, javascript/* and config/* is shared by both versions.

Hence that file is not 'PHP' related, so you can't use any PHP variables there :) When you change the web_prefix, you'll have to update the config/container.js file too i'm afraid. 

> web_prefix is not used in container.js
> --------------------------------------
>
>                 Key: SHINDIG-420
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-420
>             Project: Shindig
>          Issue Type: Bug
>          Components: Features (Javascript), OpenSocial  feature (Javascript)
>         Environment: all
>            Reporter: Erel Segal
>            Assignee: Chris Chabot
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In shindig/config/container.js, the web_prefix setting is not used.
> For example, in "opensocial-0.8", there is a line:
>     "path" : "/social/data"
> It should include the web_prefix, otherwise the samples don't work.
> I don't know how to add it, since it is defined in PHP!
> If this requires the user to change something in a JS file, I think the documentation here:
>    http://incubator.apache.org/shindig/#php
> should be updated as well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.