[6.0] base_contact "DROP LANGUAGE IF EXISTS plpgsql CASCADE;"

Bug #683761 reported by Ferdinand
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Low
OpenERP R&D Addons Team 1

Bug Description

IMHO this line should be eliminated - other software installed in the same db might need plpgsql and will not work without notice
there is no harm having the language installed.

Related branches

Revision history for this message
xrg (xrg) wrote : Re: [Bug 683761] [NEW] [6.0] base_contact "DROP LANGUAGE IF EXISTS plpgsql CASCADE; "

On Wednesday 01 December 2010, you wrote:
> Public bug reported:
>
> IMHO this line should be eliminated - other software installed in the same
> db might need plpgsql and will not work without notice there is no harm
> having the language installed.
>

I will nominate that part of the code for a 5* (five-star) on the WTF scale...

In general, I do like SQL and doing things with it (as opposed to python or
Ruby on Rails over Java over Xen etc.) . But that snippet there is just a very
bad piece of code.

Changed in openobject-addons:
status: New → Triaged
Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 2 (openerp-dev-addons2)
importance: Undecided → Low
status: Triaged → Confirmed
Changed in openobject-addons:
assignee: OpenERP R&D Addons Team 2 (openerp-dev-addons2) → OpenERP R&D Addons Team 1 (openerp-dev-addons1)
Changed in openobject-addons:
status: Confirmed → Fix Committed
status: Fix Committed → Fix Released
Revision history for this message
Ujjvala Collins (uco-openerp) wrote :

Hello,

The fix has been committed to lp:~openerp-dev/openobject-addons/trunk-dev addons1 with revision 4620.

Soon it will be merged into main addons.

Thanks

Changed in openobject-addons:
milestone: none → 6.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.