url widget should tolerate missing protocol

Bug #907337 reported by Mayur Maheshwari(OpenERP)
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Web (MOVED TO GITHUB)
Fix Released
Wishlist
OpenERP's Framework R&D

Bug Description

fields using widget="url" should tolerate missing protocol at input or display (to determine) currently they just include the value verbatim in a href="value".

Original description:
-------------------------
In Customer form view sales & purchase teb
Here one field is website and and shoort key to open a website
when i press that (^) key i got a traceback on client side and not open that website

traceback
=========
Error showing url: Error stating file '/home/mma/home/workplace/rdtools/client/trunk/bin/www.agrolait.com': No such file or directory

Related branches

Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :

Hello Mayur Maheshwari,

Would you please try with : http://www.google.com.

Thanks and waiting for your reply.

Changed in openobject-addons:
status: New → Incomplete
Revision history for this message
Mayur Maheshwari(OpenERP) (mma-openerp) wrote :

Hello

yes by putting http: //www.agrolait.com it's working Good

Thanks .

Changed in openobject-addons:
status: Incomplete → Invalid
Revision history for this message
Mayur Maheshwari(OpenERP) (mma-openerp) wrote :

Hello vishal ,

Here http://www.agrolait.com it's working but in all partners demo data load like this [www.agrolait.com ]
so I think it's improvement in demo data in all partner by http:// for avoid a client side error

Therefore, I set a status new

Thanks and waiting for your reply.

Changed in openobject-addons:
status: Invalid → New
affects: openobject-addons → openobject-server
Changed in openobject-server:
assignee: nobody → OpenERP's Framework R&D (openerp-dev-framework)
importance: Undecided → Low
status: New → Confirmed
Changed in openobject-server:
status: Confirmed → In Progress
Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :

Hello Mayur Maheshwari,

It's minor issue so I have suggested would you please make merge proposal and link your branch with this issue.

Thanks.

Revision history for this message
Mayur Maheshwari(OpenERP) (mma-openerp) wrote :

It has been fixed in lp:~openerp-dev/openobject-server/trunk-bug-907337-mma
Revision ID: <email address hidden>
Revision num: 3908

It will be available in trunk soon.

Changed in openobject-server:
status: In Progress → Fix Committed
Revision history for this message
Antony Lesuisse (OpenERP) (al-openerp) wrote :

Url widget should tolerate missing protocols but this belong to openerp-web.

Changed in openobject-server:
importance: Low → Wishlist
status: Fix Committed → Confirmed
affects: openobject-server → openerp-web
summary: - not working short key of url for open a website
+ url widget should tolerate missing protocol
description: updated
Revision history for this message
Antony Lesuisse (OpenERP) (al-openerp) wrote :

2221 revid:<email address hidden>

Changed in openerp-web:
status: Confirmed → Fix Released
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.