OpenERP Addons (modules)

Functionnal regression in "city" (extra-addons)

Reported by Numérigraphe on 2011-11-22
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenERP Addons
Undecided
Numérigraphe

Bug Description

A few improvements that I had made for the module "city" in the extra addons in revno 3761 have been lost when the module was overwritten with an alternate version in revno 5240.

From the bzr log :
* In the module "city":
      Made the "state" field optional and added a new field "country"
      Added a field "code".
      Replaced the method "search" of city.city by a simpler "name_search" with a default limit of 80 matches
      Added the country in the city list view
      Bumped version to 1.1
      Fixed a in the module's description.
      Removed the double "replace" section in the view XML (doesn't seem needed on 5.0.0)
* Enhancements in the module "city":
Added search methods for the function fields in partner.address
Indexed the location in partner.address
Reformated the code a bit
Made the city code searchable in advanced search
Also completed base_contact_city's description.

Lionel Sausin

Numérigraphe (numerigraphe) wrote :

I'll see if I can re-introduce my changes into the "new" city module.
Lionel

Amit Parik (amit-parik) on 2011-11-23
Changed in openobject-addons:
assignee: nobody → Numérigraphe (numerigraphe)
status: New → Confirmed
Numérigraphe (numerigraphe) wrote :

Dear Jordi Esteve,
I re-introduced my changes to the module "city", and by respect for your work I've made it in a distinct branch with a merge proposal.
Would you please be so kind as to review this proposal ?
Lionel.

Changed in openobject-addons:
status: Confirmed → Fix Committed
Numérigraphe (numerigraphe) wrote :

I merged my own proposal, only hoping it won't get overwritten again...
Lionel.

Changed in openobject-addons:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers