You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2017/04/30 09:54:58 UTC
svn commit: r1793254 -
/ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/FindGeneric.groovy
Author: jleroux
Date: Sun Apr 30 09:54:58 2017
New Revision: 1793254
URL: http://svn.apache.org/viewvc?rev=1793254&view=rev
Log:
Improved: When searched from FindGeneric page it looses entityName from the url
(OFBIZ-9341)
Steps to generate:
1. Go to webtools (https://localhost:8443/webtools/control/main)
2. Click on "Entity Engine" (https://localhost:8443/webtools/control/entitymaint)
3. Click on any entity to view (https://localhost:8443/webtools/control/FindGeneric?entityName=AccommodationClass)
4. Click on "search" button (https://localhost:8443/webtools/control/FindGeneric)
When the search is hit, it looses the entityName from the url as it used to
happen in previous releases and according to the pattern on other search pages.
Thanks: Aditya Sharma
Modified:
ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/FindGeneric.groovy
Modified: ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/FindGeneric.groovy
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/FindGeneric.groovy?rev=1793254&r1=1793253&r2=1793254&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/FindGeneric.groovy (original)
+++ ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/FindGeneric.groovy Sun Apr 30 09:54:58 2017
@@ -44,11 +44,10 @@ if (modelEntity) {
ModelReader entityModelReader = delegator.getModelReader()
//create the search form with auto-fields-entity
String dynamicAutoEntityFieldSearchForm = '<?xml version="1.0" encoding="UTF-8"?><forms xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://ofbiz.apache.org/Widget-Form" xsi:schemaLocation="http://ofbiz.apache.org/Widget-Form http://ofbiz.apache.org/dtds/widget-form.xsd">' +
- '<form name="FindGeneric" type="single" target="FindGeneric">' +
+ '<form name="FindGeneric" type="single" target="FindGeneric?entityName='+ entityName +'">' +
'<auto-fields-entity entity-name="' + entityName + '" default-field-type="find"/>' +
'<field name="noConditionFind"><hidden value="Y"/></field>' +
'<field name="searchOptions_collapsed" ><hidden value="true"/></field>' +
- '<field name="entityName"><hidden value="' + entityName + '"/></field>' +
'<field name="searchButton" title="' + UtilProperties.getMessage("CommonUiLabels", "FormFieldTitle_searchButton", locale) + '"><submit/></field>'
//call modelEntity to complete information on the field type
Re: svn commit: r1793254 - /ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/FindGeneric.groovy
Posted by Deepak Dixit <de...@hotwaxsystems.com>.
Thanks Taher, totally make sense
Here is the ticket for reference, I'll review and try to update this
dynamic thing
https://issues.apache.org/jira/browse/OFBIZ-9458
Thanks & Regards
--
Deepak Dixit
www.hotwaxsystems.com
www.hotwax.co
On Sun, Apr 30, 2017 at 5:34 PM, Taher Alkhateeb <slidingfilaments@gmail.com
> wrote:
> Whoa! That was painful to read, this file It is creating dynamic models for
> XML documents and then parsing them for stuff like entities and widgets and
> hard-coding XML attributes and then shoving everything back to the context.
> Phew!
>
> If anyone has the time and would like to review this, I would recommend
> re-writing this thing completely from scratch with all dependencies
> updated.
>
> On Sun, Apr 30, 2017 at 12:54 PM, <jl...@apache.org> wrote:
>
> > Author: jleroux
> > Date: Sun Apr 30 09:54:58 2017
> > New Revision: 1793254
> >
> > URL: http://svn.apache.org/viewvc?rev=1793254&view=rev
> > Log:
> > Improved: When searched from FindGeneric page it looses entityName from
> > the url
> > (OFBIZ-9341)
> >
> > Steps to generate:
> > 1. Go to webtools (https://localhost:8443/webtools/control/main)
> > 2. Click on "Entity Engine" (https://localhost:8443/
> > webtools/control/entitymaint)
> > 3. Click on any entity to view (https://localhost:8443/
> > webtools/control/FindGeneric?entityName=AccommodationClass)
> > 4. Click on "search" button (https://localhost:8443/
> > webtools/control/FindGeneric)
> >
> > When the search is hit, it looses the entityName from the url as it used
> to
> > happen in previous releases and according to the pattern on other search
> > pages.
> >
> > Thanks: Aditya Sharma
> >
> > Modified:
> > ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/
> > FindGeneric.groovy
> >
> > Modified: ofbiz/ofbiz-framework/trunk/framework/webtools/
> > groovyScripts/entity/FindGeneric.groovy
> > URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/
> > framework/webtools/groovyScripts/entity/FindGeneric.groovy?rev=
> > 1793254&r1=1793253&r2=1793254&view=diff
> > ============================================================
> > ==================
> > --- ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/
> FindGeneric.groovy
> > (original)
> > +++ ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/
> FindGeneric.groovy
> > Sun Apr 30 09:54:58 2017
> > @@ -44,11 +44,10 @@ if (modelEntity) {
> > ModelReader entityModelReader = delegator.getModelReader()
> > //create the search form with auto-fields-entity
> > String dynamicAutoEntityFieldSearchForm = '<?xml version="1.0"
> > encoding="UTF-8"?><forms xmlns:xsi="http://www.w3.org/
> > 2001/XMLSchema-instance" xmlns="http://ofbiz.apache.org/Widget-Form"
> > xsi:schemaLocation="http://ofbiz.apache.org/Widget-Form
> > http://ofbiz.apache.org/dtds/widget-form.xsd">' +
> > - '<form name="FindGeneric" type="single"
> > target="FindGeneric">' +
> > + '<form name="FindGeneric" type="single" target="FindGeneric?
> entityName='+
> > entityName +'">' +
> > '<auto-fields-entity entity-name="' + entityName + '"
> > default-field-type="find"/>' +
> > '<field name="noConditionFind"><hidden value="Y"/></field>'
> +
> > '<field name="searchOptions_collapsed" ><hidden
> > value="true"/></field>' +
> > - '<field name="entityName"><hidden value="' + entityName +
> > '"/></field>' +
> > '<field name="searchButton" title="' +
> > UtilProperties.getMessage("CommonUiLabels",
> > "FormFieldTitle_searchButton", locale) + '"><submit/></field>'
> >
> > //call modelEntity to complete information on the field type
> >
> >
> >
>
Re: svn commit: r1793254 - /ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/FindGeneric.groovy
Posted by Taher Alkhateeb <sl...@gmail.com>.
Whoa! That was painful to read, this file It is creating dynamic models for
XML documents and then parsing them for stuff like entities and widgets and
hard-coding XML attributes and then shoving everything back to the context.
Phew!
If anyone has the time and would like to review this, I would recommend
re-writing this thing completely from scratch with all dependencies updated.
On Sun, Apr 30, 2017 at 12:54 PM, <jl...@apache.org> wrote:
> Author: jleroux
> Date: Sun Apr 30 09:54:58 2017
> New Revision: 1793254
>
> URL: http://svn.apache.org/viewvc?rev=1793254&view=rev
> Log:
> Improved: When searched from FindGeneric page it looses entityName from
> the url
> (OFBIZ-9341)
>
> Steps to generate:
> 1. Go to webtools (https://localhost:8443/webtools/control/main)
> 2. Click on "Entity Engine" (https://localhost:8443/
> webtools/control/entitymaint)
> 3. Click on any entity to view (https://localhost:8443/
> webtools/control/FindGeneric?entityName=AccommodationClass)
> 4. Click on "search" button (https://localhost:8443/
> webtools/control/FindGeneric)
>
> When the search is hit, it looses the entityName from the url as it used to
> happen in previous releases and according to the pattern on other search
> pages.
>
> Thanks: Aditya Sharma
>
> Modified:
> ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/
> FindGeneric.groovy
>
> Modified: ofbiz/ofbiz-framework/trunk/framework/webtools/
> groovyScripts/entity/FindGeneric.groovy
> URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/
> framework/webtools/groovyScripts/entity/FindGeneric.groovy?rev=
> 1793254&r1=1793253&r2=1793254&view=diff
> ============================================================
> ==================
> --- ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/FindGeneric.groovy
> (original)
> +++ ofbiz/ofbiz-framework/trunk/framework/webtools/groovyScripts/entity/FindGeneric.groovy
> Sun Apr 30 09:54:58 2017
> @@ -44,11 +44,10 @@ if (modelEntity) {
> ModelReader entityModelReader = delegator.getModelReader()
> //create the search form with auto-fields-entity
> String dynamicAutoEntityFieldSearchForm = '<?xml version="1.0"
> encoding="UTF-8"?><forms xmlns:xsi="http://www.w3.org/
> 2001/XMLSchema-instance" xmlns="http://ofbiz.apache.org/Widget-Form"
> xsi:schemaLocation="http://ofbiz.apache.org/Widget-Form
> http://ofbiz.apache.org/dtds/widget-form.xsd">' +
> - '<form name="FindGeneric" type="single"
> target="FindGeneric">' +
> + '<form name="FindGeneric" type="single" target="FindGeneric?entityName='+
> entityName +'">' +
> '<auto-fields-entity entity-name="' + entityName + '"
> default-field-type="find"/>' +
> '<field name="noConditionFind"><hidden value="Y"/></field>' +
> '<field name="searchOptions_collapsed" ><hidden
> value="true"/></field>' +
> - '<field name="entityName"><hidden value="' + entityName +
> '"/></field>' +
> '<field name="searchButton" title="' +
> UtilProperties.getMessage("CommonUiLabels",
> "FormFieldTitle_searchButton", locale) + '"><submit/></field>'
>
> //call modelEntity to complete information on the field type
>
>
>