Re: svn commit: r1140933 - in /ofbiz/trunk/applications: commonext/webapp/WEB-INF/controller.xml content/webapp/content/WEB-INF/controller.xml

[Scott Gray-2]

Hi Hans,

I'm not sure I understand, what was preventing it from being used in all components?

Thanks
Scott

On 29/06/2011, at 1:20 PM, [hidden email] wrote:

> Author: hansbak
> Date: Wed Jun 29 01:20:24 2011
> New Revision: 1140933
>
> URL: http://svn.apache.org/viewvc?rev=1140933&view=rev
> Log:
> move the stream request from content to commonext so streaming of image/video can be used in all components related to rev 1140919
>
> Modified:
>    ofbiz/trunk/applications/commonext/webapp/WEB-INF/controller.xml
>    ofbiz/trunk/applications/content/webapp/content/WEB-INF/controller.xml
>
> Modified: ofbiz/trunk/applications/commonext/webapp/WEB-INF/controller.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/commonext/webapp/WEB-INF/controller.xml?rev=1140933&r1=1140932&r2=1140933&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/commonext/webapp/WEB-INF/controller.xml (original)
> +++ ofbiz/trunk/applications/commonext/webapp/WEB-INF/controller.xml Wed Jun 29 01:20:24 2011
> @@ -35,4 +35,10 @@ under the License.
>         <response name="success" type="view-last"/>
>     </request-map>
>
> +    <request-map uri="stream">
> +        <event type="java" path="org.ofbiz.content.data.DataEvents" invoke="serveObjectData"/>
> +        <response name="success" type="none"/>
> +        <response name="error" type="view" value="error"/>
> +    </request-map>
> +    
> </site-conf>
>
> Modified: ofbiz/trunk/applications/content/webapp/content/WEB-INF/controller.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/content/webapp/content/WEB-INF/controller.xml?rev=1140933&r1=1140932&r2=1140933&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/content/webapp/content/WEB-INF/controller.xml (original)
> +++ ofbiz/trunk/applications/content/webapp/content/WEB-INF/controller.xml Wed Jun 29 01:20:24 2011
> @@ -1002,12 +1002,6 @@ under the License.
>         <response name="success" type="none"/>
>         <response name="error" type="request" value="main"/>
>     </request-map>
> -    <request-map uri="stream">
> -        <event type="java" path="org.ofbiz.content.data.DataEvents" invoke="serveObjectData"/>
> -        <response name="success" type="none"/>
> -        <response name="error" type="view" value="error"/>
> -    </request-map>
> -
>     <!-- ================ ContentOperation Requests ================= -->
>     <request-map uri="EditContentOperation"><security auth="true" https="true"/><response name="success" type="view" value="EditContentOperation"/></request-map>
>     <request-map uri="addContentOperation">
>
>

... [show rest of quote]

... [show rest of quote]

[hans_bakker]

please check revision: 1140919 where i removed the
hardcoded /content/control

Regards,
Hans

On Wed, 2011-06-29 at 13:34 +1200, Scott Gray wrote:
> 1140919
--
Ofbiz on twitter: http://twitter.com/apache_ofbiz
Myself on twitter: http://twitter.com/hansbak
Antwebsystems.com: Quality services for competitive rates.

[Scott Gray-2]

Yeah I saw that, you mentioned it in the commit message.  For the record I'm not criticizing or saying there is anything wrong with your commit, I'm just trying to understand the problem better.

Is your content app not located at somedomain.com/content? It just seems like the need to make a request available to all apps shouldn't have to involve adding that request to every app.

Thanks
Scott

On 29/06/2011, at 2:59 PM, Hans Bakker wrote:

> please check revision: 1140919 where i removed the
> hardcoded /content/control
>
> Regards,
> Hans
>
> On Wed, 2011-06-29 at 13:34 +1200, Scott Gray wrote:
>> 1140919
> --

[hans_bakker]

On Wed, 2011-06-29 at 16:07 +1200, Scott Gray wrote:
> Yeah I saw that, you mentioned it in the commit message.  For the record I'm not criticizing or saying there is anything wrong with your commit, I'm just trying to understand the problem better.
>
> Is your content app not located at somedomain.com/content? It just seems like the need to make a request available to all apps shouldn't have to involve adding that request to every app.
>

normally not required, however if you use virtual root hosting you
cannot have requests from other web applications.

at the moment i am running all these sites as virtual hosts on a single
ofbiz installation.

http://www.antwebsystems.com
http://www.ofbiz.info
http://www.ofbizshop.com
http://www.ofbiz.nl
http://www.ofbiz.in.th

Regards,
Hans

> Thanks
> Scott
>
> On 29/06/2011, at 2:59 PM, Hans Bakker wrote:
>
> > please check revision: 1140919 where i removed the
> > hardcoded /content/control
> >
> > Regards,
> > Hans
> >
> > On Wed, 2011-06-29 at 13:34 +1200, Scott Gray wrote:
> >> 1140919
> > --
>

... [show rest of quote]

--
Ofbiz on twitter: http://twitter.com/apache_ofbiz
Myself on twitter: http://twitter.com/hansbak
Antwebsystems.com: Quality services for competitive rates.