c2c-oerpscenario team mailing list archive

[Raphaël Valyi - http://www.akretion.com <745045@xxxxxxxxxxxxxxxxxx>]

Hello Guewen,
I agree with changing that by an = indeed
We just need to send a mail on the mailing list so people get warned about
it and double check.
Oh, BTW, we are completing Openbravo POS/OpenERP integration using
TerminatOOOR (new generation) + base_sale_multichannels...
Working smoothly.

On Tue, Mar 29, 2011 at 12:43 PM, Guewen Baconnier @ Camptocamp <
745045@xxxxxxxxxxxxxxxxxx> wrote:

> ** Patch added: "extra-addons v5 patch"
>
> https://bugs.launchpad.net/bugs/745045/+attachment/1953698/+files/payment_code_to_payment_settings.patch
>
> --
> You received this bug notification because you are subscribed to Magento
> Open ERP Connector.
> https://bugs.launchpad.net/bugs/745045
>
> Title:
>  base_sale_multichannels: base.sale.payment.type multiple matches
>
> Status in Magento Open ERP Connector:
>  New
>
> Bug description:
>  Hi,
>
>  This bug concerns the module base_sale_multichannels in extra-addons,
>  so I'm not sure if I better have to report the bug here or on
>  openobject-addons. Please tell me if I have to report on extra-addons
>  next time.
>
>  The payment codes in base.sale.payment.type are a list of codes
>  separated by a ; like "ccsave;free".
>
>  There is a method payment_code_to_payment_settings in the sale.order
>  class which searches and returns the right object of
>  base.sale.payment.type according to the payment_code given by magento.
>
>  The search is basically a "ilike" on the name, that's efficient and
>  works perfect with most of cases.
>
>  But it can happen that you have one payment settings "bank" and one
>  "bankcr" for example (we have this issue).
>
>  The method returns the first found, so it can return the settings of
>  "bankcr" instead of "bank".
>
>  I just added a few lines in that method which search the exact match
>  when many rows are found with the "ilike".
>
>  The attached patch is for extra-addons V5. But if you agree with my
>  fix, I can give you a patch for V6 or apply them myself to both extra-
>  addons v5 and v6. Just confirm me that's ok for you.
>
>  Thanks
>  Guewen
>

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
https://bugs.launchpad.net/bugs/745045

Title:
  base_sale_multichannels: base.sale.payment.type multiple matches

Status in Magento Open ERP Connector:
  New

Bug description:
  Hi,

  This bug concerns the module base_sale_multichannels in extra-addons,
  so I'm not sure if I better have to report the bug here or on
  openobject-addons. Please tell me if I have to report on extra-addons
  next time.

  The payment codes in base.sale.payment.type are a list of codes
  separated by a ; like "ccsave;free".

  There is a method payment_code_to_payment_settings in the sale.order
  class which searches and returns the right object of
  base.sale.payment.type according to the payment_code given by magento.

  The search is basically a "ilike" on the name, that's efficient and
  works perfect with most of cases.

  But it can happen that you have one payment settings "bank" and one
  "bankcr" for example (we have this issue).

  The method returns the first found, so it can return the settings of
  "bankcr" instead of "bank".

  I just added a few lines in that method which search the exact match
  when many rows are found with the "ilike".

  The attached patch is for extra-addons V5. But if you agree with my
  fix, I can give you a patch for V6 or apply them myself to both extra-
  addons v5 and v6. Just confirm me that's ok for you.

  Thanks
  Guewen