[Openerp-community] [Merge] lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes into lp:~openerp-community/openobject-addons/extra-trunk-esale_joomla

classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

[Openerp-community] [Merge] lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes into lp:~openerp-community/openobject-addons/extra-trunk-esale_joomla

Stefan Rijnhart (Therp)
Stefan Rijnhart (Therp) has proposed merging lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes into lp:~openerp-community/openobject-addons/extra-trunk-esale_joomla.

Requested reviews:
  Lorenzo Battistini - Agile BG - Domsense (elbati)

For more details, see:
https://code.launchpad.net/~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes/+merge/87799

Hi Lorenzo,

thank you for your work on the esale_joomla module. Please consider these two additional fixes for getting this module to work under OpenERP 6.0, so that we can keep your branch as a central point for this module.

- Do not refer to the missing field 'rental' on the product form
- Fix the product view on Products by Web Category

Kind regards,
Stefan.

--
https://code.launchpad.net/~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes/+merge/87799
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes.

=== modified file 'esale_joomla/esale_joomla_view.xml'
--- esale_joomla/esale_joomla_view.xml 2011-02-25 09:23:22 +0000
+++ esale_joomla/esale_joomla_view.xml 2012-01-06 18:57:24 +0000
@@ -515,7 +515,7 @@
         <field name="model">product.product</field>
         <field name="inherit_id" ref="view_product_book_form"/>
         <field name="arch" type="xml">
-            <field name="rental" position="after">
+            <field name="purchase_ok" position="after">
                 <field name="online" select="2"/>
             </field>
         </field>
@@ -539,6 +539,22 @@
         <field name="view_type">tree</field>
         <field name="view_id" ref="esale_joomla_category_tree_view"/>
     </record>
+    <record id="esale_product_action_tree" model="ir.actions.act_window">
+        <field name="name">Products</field>
+        <field name="type">ir.actions.act_window</field>
+        <field name="res_model">product.product</field>
+        <field name="view_type">form</field>
+        <field name="view_mode">tree,form</field>
+        <field name="domain">[('esale_category_ids','=',active_id)]</field>
+        <field name="context">{'esale_category_ids':active_id}</field>
+    </record>
+    <record id="ir_esale_product_category_open" model="ir.values">
+        <field name="key2">tree_but_open</field>
+        <field name="model">esale_joomla.category</field>
+        <field name="name">Products by Web Category</field>
+        <field eval="'ir.actions.act_window,%d'%esale_product_action_tree" name="value"/>
+        <field eval="True" name="object"/>
+    </record>
     <menuitem action="esale_joomla_category_action" id="menu_product_category_action" parent="base.menu_product" sequence="1"/>
 
     <record id="esale_joomla_category_action_form" model="ir.actions.act_window">


_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [hidden email]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp
Reply | Threaded
Open this post in threaded view
|

[Openerp-community] [Merge] lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes into lp:~openerp-community/openobject-addons/extra-trunk-esale_joomla

noreply
The proposal to merge lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes into lp:~openerp-community/openobject-addons/extra-trunk-esale_joomla has been updated.

    Status: Needs review => Merged

For more details, see:
https://code.launchpad.net/~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes/+merge/87799
--
https://code.launchpad.net/~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes/+merge/87799
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [hidden email]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp
Reply | Threaded
Open this post in threaded view
|

Re: [Openerp-community] [Merge] lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes into lp:~openerp-community/openobject-addons/extra-trunk-esale_joomla

eLBatti
In reply to this post by Stefan Rijnhart (Therp)
Hello Stefan, thanks for your contribution.
Actually, I have not used esale_joomla for a long time.
If you confirm it works on OpenERP 6.0.*, we could merge it into lp:openobject-addons/extra-6.0 .
What do you think?
--
https://code.launchpad.net/~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes/+merge/87799
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [hidden email]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp
Reply | Threaded
Open this post in threaded view
|

Re: [Openerp-community] [Merge] lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes into lp:~openerp-community/openobject-addons/extra-trunk-esale_joomla

Stefan Rijnhart (Therp)
In reply to this post by Stefan Rijnhart (Therp)
Hi Lorenzo,

thanks for the merge! Yes, we should definitely merge into extra-6.0. I have two more fixes to test and upload:

- In the plugin, retrieve language 'iso' column, not 'shortcode' to prevent 'Cannot match language' problem during XML-RPC communication (the plugin reports language 'en' which does not match 'en_US' on the OpenERP side).
- Revise permissions (which are currently not loaded)

The second issue is quite straightforward, but I was wondering about the first: did you come across the first problem and did you try something similar? Cannot say yet whether our fix will actually work, as our customer's Joomla administrator does not work weekends ;-)

Cheers,
Stefan.

--
https://code.launchpad.net/~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes/+merge/87799
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [hidden email]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp
Reply | Threaded
Open this post in threaded view
|

Re: [Openerp-community] [Merge] lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes into lp:~openerp-community/openobject-addons/extra-trunk-esale_joomla

eLBatti
>
> - In the plugin, retrieve language 'iso' column, not 'shortcode' to prevent 'Cannot match language' problem during XML-RPC communication (the plugin reports language 'en' which does not match 'en_US' on the OpenERP side).
> - Revise permissions (which are currently not loaded)
>
> The second issue is quite straightforward, but I was wondering about the first: did you come across the first problem and did you try something similar? Cannot say yet whether our fix will actually work, as our customer's Joomla administrator does not work weekends ;-)

Uhm, you are right, I remember I had to edit some country code on
joomfish configuration.
If joomfish is still a dependency, we should mention it somewhere. I
think it would be good if we put a sort of quick installation guide
within the __openerp__.py

--
https://code.launchpad.net/~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes/+merge/87799
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [hidden email]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp
Reply | Threaded
Open this post in threaded view
|

Re: [Openerp-community] [Merge] lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes into lp:~openerp-community/openobject-addons/extra-trunk-esale_joomla

Daniel Stenlöv (XCLUDE)
In reply to this post by Stefan Rijnhart (Therp)
If you want to support multiple languages in the VirtueMart/Joomla system you will need to install joomfish. But if you are just using 1 language in your VM/J installation this isn't needed even if you do have multiple languages in your OpenERP installation. You will however need to have the same languages in both system.

But there should be written somewhere that the xmlrpc plugin needs to be installed on the Joomla system. Easiest way to do this is to pack the included plugin into a zip file and then install it as a normal Joomla plugin.

Of my knowledge the OpenERP Joomla plugin doesn't support multiple stores or changed database path, the path need to be vm at this moment. I have been thinking of fixing this.

The plugin uses the database handler from Joomla, but it should really use the handler from VirtueMart or even better use the functions in VirtueMart. The plugin is badly in need of some love.

/Best wishes, Daniel
--
https://code.launchpad.net/~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes/+merge/87799
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [hidden email]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp
Reply | Threaded
Open this post in threaded view
|

Re: [Openerp-community] [Merge] lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes into lp:~openerp-community/openobject-addons/extra-trunk-esale_joomla

Daniel Stenlöv (XCLUDE)
In reply to this post by Stefan Rijnhart (Therp)
I just noticed that some of the queries is does also need joomla to use the name jos.

For example tax:
SELECT tax_rate FROM jos_vm_tax_rate where tax_rate_id = (SELECT shipping_rate_vat_id FROM jos_vm_shipping_rate where shipping_rate_id = ".$order_shipping_rate_id.")
           
--
https://code.launchpad.net/~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes/+merge/87799
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/extra-6.0-esale-joomla-view-fixes.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [hidden email]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp