You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ha...@apache.org on 2011/06/29 03:20:25 UTC

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

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">



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

Posted by Hans Bakker <ma...@antwebsystems.com>.
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
> > -- 
> 

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


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

Posted by Scott Gray <sc...@hotwaxmedia.com>.
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
> -- 


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

Posted by Hans Bakker <ma...@antwebsystems.com>.
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.


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

Posted by Scott Gray <sc...@hotwaxmedia.com>.
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, hansbak@apache.org 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">
> 
>