[jira] Created: (OFBIZ-1372) Enhance the integration with Google Product

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
26 messages Options
12
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
Enhance the integration with Google Product
-------------------------------------------

                 Key: OFBIZ-1372
                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
             Project: OFBiz
          Issue Type: Improvement
    Affects Versions: SVN trunk
         Environment: mac os x
            Reporter: Marco Risaliti
            Priority: Minor
             Fix For: SVN trunk


This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.

Thanks
Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marco Risaliti updated OFBIZ-1372:
----------------------------------

    Attachment: google.patch

This is the patch for removing the google base integration from the product application.



> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marco Risaliti updated OFBIZ-1372:
----------------------------------

    Attachment: googlebase.zip

This is the new google base application to be insert into the specialpurpose folder.

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12538224 ]

Marco Risaliti commented on OFBIZ-1372:
---------------------------------------

The method ProductSearchEvents.searchExportProductListToGoogle is still in the product application and I have to check if has to be moved or not.
In this moment I have removed the left side of advanced search into the catalog in the new google base application.
And still remain to understand hwo to handle the permission for this new application.
Please let me know your ideas before continue and finish this new implementation.

Thanks
Marco

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12538234 ]

Jacopo Cappellato commented on OFBIZ-1372:
------------------------------------------

Marco,

looks great.
About the permission, I'd suggest to start in a very simple way: we could add a new GOOGLEBASE_VIEW permission (the seed data should be in a file named data/GooglebaseSecurityData.xml) and add GOOGLEBASE to the base permission attribute in the ofbiz-controller.xml:

base-permission="OFBTOOLS,GOOGLEBASE"

For an example, have a look at the Oagis component.

It would be great to move the ProductSearchEvents.searchExportProductListToGoogle method into a new class in the googlebase component.

By the way, what if we commit this first step?

Thanks,

Jacopo


> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Assigned: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacopo Cappellato reassigned OFBIZ-1372:
----------------------------------------

    Assignee: Jacopo Cappellato

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12538264 ]

Marco Risaliti commented on OFBIZ-1372:
---------------------------------------

Hi Jacopo,

I agree with your suggestions and  I can do those changes on the next days.
In the same moment if you want to commit this first step for me it's ok.

Thanks
Marco

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12538297 ]

Jacopo Cappellato commented on OFBIZ-1372:
------------------------------------------

Comment your work (plus mine) is in rev. 589315 and 589317
I've used different approaches from yours but the end result should be the same.
Thanks!

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marco Risaliti updated OFBIZ-1372:
----------------------------------

    Attachment: google_2.patch

This is the second patch that implements:
- the security for the google base application
- the method searchExportProductListToGoogle is moved in the new class GoogleBaseSearchEvents

Thanks
Marco

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12538347 ]

Marco Risaliti commented on OFBIZ-1372:
---------------------------------------

Hi Jacopo,
 
also I prefer your different approaches on keywordsearch that now is more reusable in different places.
I have also two new ideas :

- instead of looking for GoodIdentification for a google product only if one specify insert in the form we can che everytime if the product is in GoodIdentification then for us is an update.

String googleProductId = null;
                     if (!"insert".equals(actionType)) {
                         try {
                             GenericValue googleProduct = delegator.findByPrimaryKey("GoodIdentification", UtilMisc.toMap("productId", prod.getString("productId"), "goodIdentificationTypeId", "GOOGLE_ID"));
                             if (UtilValidate.isNotEmpty(googleProduct)) {
                                googleProductId = googleProduct.getString("idValue");
                             }
                         } catch(GenericEntityException gee) {
                             Debug.logError("Unable to get the Google id for product [" + prod.getString("productId") + "]: " + gee.getMessage(), module);
                         }
                     }
                     if ("update".equals(actionType) && UtilValidate.isEmpty(googleProductId)) {
                         itemActionType = "insert";
                     }


String googleProductId = null;
try {
                             GenericValue googleProduct = delegator.findByPrimaryKey("GoodIdentification", UtilMisc.toMap("productId", prod.getString("productId"), "goodIdentificationTypeId", "GOOGLE_ID"));
                             if (UtilValidate.isNotEmpty(googleProduct)) {
                                googleProductId = googleProduct.getString("idValue");
                             }
                         } catch(GenericEntityException gee) {
                             Debug.logError("Unable to get the Google id for product [" + prod.getString("productId") + "]: " + gee.getMessage(), module);
                         }
}

so if googleProductId is not null then we will send an update on google.
Because now if you have already the product sent to google and you send again the same product with an insert action you will have double products on google.

- Why don't we add a new tab in the google base application to show which products are sent to google base reading the table GoodIdentification with code GOOGLE_ID ?

Thanks
Marco


> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12538405 ]

Jacopo Cappellato commented on OFBIZ-1372:
------------------------------------------

Thanks Marco,

your patch is in rev. 589499

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12538407 ]

Jacopo Cappellato commented on OFBIZ-1372:
------------------------------------------

Marco,

I agree with you about the googleProductId: what you propose is the right way to go.

I also like the idea of having a new screen to list products sent to Google: however we will probably need a search screen as well because the number of products sent to google could be very high: we could probably reuse (again) the advanced search screen (and hiding somewhere the condition on GOOGLE_ID not null)... just my 2 cents.


> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marco Risaliti updated OFBIZ-1372:
----------------------------------

    Attachment: googlebase_new3.patch

That's a new patch that includes a new feature to search products by Good Identification Type/Value and so now it can be used also to see which products are exported to Google Base.
Then I have tried to applied the new style sheets to this new application.

Thanks
Marco

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip, googlebase_new3.patch
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12543810 ]

Jacopo Cappellato commented on OFBIZ-1372:
------------------------------------------

Marco, it looks really interesting.

My only concern is about the new style:

- we should avoid to insert html specific code into the screen definition (they should be as much html independent as possible); could you please
- would it be possible to provide a patch without layout changes? From the patch it is very difficult to understand what has been modified in the advancedsearch.ftl and keywordsearch.ftl templates

Thanks

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip, googlebase_new3.patch
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12543811 ]

Marco Risaliti commented on OFBIZ-1372:
---------------------------------------

Hi Jacopo,

also I didn't like to insert this type of code into the screen definition:

<label text="&lt;table&gt;&lt;tr&gt;&lt;td style=&quot;head3&quot;&gt;&lt;b&gt;${uiLabelMap.PageTitleGoogleBaseProductsExportToGoogle}&lt;/b&gt;&lt;/td&gt;&lt;/tr&gt;&lt;/table&gt;"/>

but this is the only solution I have found for the moment.
The issue is due that if I use simple the label tag it uses the span html tag and then the layout is not correct.
<ul>
    <li class="head3">${uiLabelMap.PageTitleGoogleBaseProductsExportToGoogle}</li>
</ul>
<br class="clear" />

Probably we can implement a table tag into the screen widget or a different solution that I didn't know.

For the advancedsearch.ftl and keywordsearch.ftl I have reformatted all the source because it was unreadable to me and I have follow the 4 spaces indentation.
By the way apart the reformatting I have simple change the header/the footer of the form and add two new fields (Good Identification Type/Value).

Thanks
Marco

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip, googlebase_new3.patch
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12543820 ]

Jacopo Cappellato commented on OFBIZ-1372:
------------------------------------------

I see...
well, I don't think we should even use html tables for to implement the screen layout but, at the moment, I don't know what is the best solution.
Hopefully one of the layout gurus (Adrian?) out there will provide a css solution :-)


> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip, googlebase_new3.patch
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Adrian Crum updated OFBIZ-1372:
-------------------------------

    Attachment: googlebase_new3.patch

Attached patch is the same as the original, but I updated it to the latest SVN and I eliminated deprecated styles from the HTML.

The previous/next navigation table could be replaced by an unordered list and placed in the screenlet title bar.

The ugly <label> element in specialpurpose/googlebase/widget/GoogleBaseScreens.xml could be eliminated if the patch in OFBIZ-875 was committed. ;-)


> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip, googlebase_new3.patch, googlebase_new3.patch
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marco Risaliti updated OFBIZ-1372:
----------------------------------

    Attachment: googlebase_new4.patch

Hi Jacopo & Adrian,

This patch replace all the others.
I have followed the suggestion by Adrian in advancedSearch.ftl about wrong UI in the screen title bar and now it's better.
Then about the main issue for the screen widget I have found a different solution all base into the main.css without adding the patch into the issue OFBIZ-875.
I have simply insert a new head4 tag into the main.css and used into the screenwidget GoogleBaseScreens.xml and now the <label> element is not more ugly and now is more readable.
Please let me know what you and the others developer think about it.

Thanks
Marco

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip, googlebase_new3.patch, googlebase_new3.patch, googlebase_new4.patch
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12545225 ]

Adrian Crum commented on OFBIZ-1372:
------------------------------------

Marco,

It would help if your work started with the patch I supplied. Your latest patch includes the deprecated styles I eliminated in my patch.


> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip, googlebase_new3.patch, googlebase_new3.patch, googlebase_new4.patch
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-1372) Enhance the integration with Google Product

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12545279 ]

Marco Risaliti commented on OFBIZ-1372:
---------------------------------------

Hi Adrian,

I have seen only this comment in the patch you provide:
+        <h3>${uiLabelMap.ProductAdvancedSearchinCategory}</h3>
+        <#-- Commented out the Find button for now. For some reason it whacks out the
+        layout - even though it shouldn't -->
+        <#-- <ul>
+            <li class="head3">${uiLabelMap.ProductAdvancedSearchinCategory}</li>
+            <li><a href="javascript:document.advtokeywordsearchform.submit()">${uiLabelMap.CommonFind}</a></li>
+        </ul>
+        <br class="clear"/> -->

and I have include it without the commented code, can you give an example so I can understand what you mean ?

Thanks
Marco

> Enhance the integration with Google Product
> -------------------------------------------
>
>                 Key: OFBIZ-1372
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1372
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: SVN trunk
>         Environment: mac os x
>            Reporter: Marco Risaliti
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: google.patch, google_2.patch, googlebase.zip, googlebase_new3.patch, googlebase_new3.patch, googlebase_new4.patch
>
>
> This is the first patch for implementing the new Google Base application, this job is still not completed it's only a preview if someone wants to check it.
> Thanks
> Marco

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

12