Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0002379Openbravo ERPC. Securitypublic2007-12-17 21:562009-10-06 11:16
villind 
shuehner 
urgentminoralways
closedfixed 
5
2.40 
2.50MP6 
Core
No
0002379: Changing pass for un-saved user changes it to current user
If you are creating a user and try to enter password it will update password for the current user.

250MP1
depends on backport 0003629 closed alostale Changing pass for un-saved user changes it to cu 
depends on backport 0004942 closed shuehner Changing pass for un-saved user changes it to current user 
Issue History
2008-07-03 19:33plujanStatusresolved => new
2008-07-03 19:33plujanResolutionfixed => open
2008-07-03 19:33plujanNote Added: 0008163
2008-07-03 19:33plujanFixed in Version2.40alpha-r2 =>
2008-07-04 09:03cromeroStatusnew => scheduled
2008-07-04 09:03cromerofix_in_branch => trunk
2008-12-10 09:58alostaleAssigned Toalostale => iperdomo
2009-01-19 10:25rafarodaversion => 2.40
2009-01-19 10:25rafarodafix_in_branchtrunk =>
2009-01-19 10:25rafarodaDescription Updated
2009-02-03 17:49iperdomoAssigned Toiperdomo => shuehner
2009-02-09 14:09pjuvaraPrioritynormal => high
2009-02-09 14:09pjuvaraNote Added: 0013267
2009-02-10 14:09shuehnerRegression testing => No
2009-02-10 14:09shuehnerStatusscheduled => closed
2009-02-10 14:09shuehnerNote Added: 0013309
2009-02-10 14:09shuehnerResolutionopen => no change required
2009-02-10 17:57plujanStatusclosed => new
2009-02-10 17:57plujanResolutionno change required => open
2009-02-10 17:57plujanNote Added: 0013327
2009-02-24 12:00psarobeStatusnew => acknowledged
2009-02-24 12:00psarobeStatusacknowledged => scheduled
2009-02-24 12:00psarobefix_in_branch => trunk
2009-04-23 16:05psarobeTag Attached: 250MP1
2009-08-02 19:50psarobePriorityhigh => urgent
2009-08-02 19:50psarobefix_in_branchpi =>
2009-09-11 19:57hgbotCheckin
2009-09-11 19:57hgbotNote Added: 0019818
2009-09-11 19:57hgbotStatusscheduled => resolved
2009-09-11 19:57hgbotResolutionopen => fixed
2009-09-11 19:57hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/c70b711ebf99bbf7f7e0842566299d7145156ada [^]
2009-09-11 20:25hudsonbotCheckin
2009-09-11 20:25hudsonbotNote Added: 0019819
2009-09-11 20:31hudsonbotCheckin
2009-09-11 20:31hudsonbotNote Added: 0019820
2009-09-11 20:50hudsonbotCheckin
2009-09-11 20:50hudsonbotNote Added: 0019821
2009-09-11 20:51hudsonbotCheckin
2009-09-11 20:51hudsonbotNote Added: 0019822
2009-09-11 21:00hudsonbotCheckin
2009-09-11 21:00hudsonbotNote Added: 0019823
2009-09-11 21:17hudsonbotCheckin
2009-09-11 21:17hudsonbotNote Added: 0019832
2009-09-11 21:24hudsonbotCheckin
2009-09-11 21:24hudsonbotNote Added: 0019833
2009-09-11 23:15hudsonbotCheckin
2009-09-11 23:15hudsonbotNote Added: 0019848
2009-09-12 09:53hudsonbotCheckin
2009-09-12 09:53hudsonbotNote Added: 0019852
2009-09-12 21:17hudsonbotCheckin
2009-09-12 21:17hudsonbotNote Added: 0019861
2009-09-13 09:54hudsonbotCheckin
2009-09-13 09:54hudsonbotNote Added: 0019862
2009-09-13 11:22hudsonbotCheckin
2009-09-13 11:22hudsonbotNote Added: 0019863
2009-09-13 21:17hudsonbotCheckin
2009-09-13 21:17hudsonbotNote Added: 0019874
2009-09-14 10:39hudsonbotCheckin
2009-09-14 10:39hudsonbotNote Added: 0019880
2009-09-14 11:51hudsonbotCheckin
2009-09-14 11:51hudsonbotNote Added: 0019897
2009-09-14 14:54hudsonbotCheckin
2009-09-14 14:54hudsonbotNote Added: 0019911
2009-09-14 16:54hudsonbotCheckin
2009-09-14 16:54hudsonbotNote Added: 0019914
2009-09-14 18:37hudsonbotCheckin
2009-09-14 18:37hudsonbotNote Added: 0019922
2009-09-14 19:57hudsonbotCheckin
2009-09-14 19:57hudsonbotNote Added: 0019947
2009-09-14 21:07hudsonbotCheckin
2009-09-14 21:07hudsonbotNote Added: 0019971
2009-09-14 21:17hudsonbotCheckin
2009-09-14 21:17hudsonbotNote Added: 0019979
2009-09-14 23:15hudsonbotCheckin
2009-09-14 23:15hudsonbotNote Added: 0020001
2009-09-15 01:09hudsonbotCheckin
2009-09-15 01:09hudsonbotNote Added: 0020013
2009-09-15 12:12hudsonbotCheckin
2009-09-15 12:12hudsonbotNote Added: 0020049
2009-09-15 12:35hudsonbotCheckin
2009-09-15 12:35hudsonbotNote Added: 0020062
2009-09-18 11:35sureshbabuStatusresolved => closed
2009-09-18 11:35sureshbabuNote Added: 0020191
2009-09-18 11:35sureshbabuFixed in Version => 2.50MP6
2009-09-18 12:38psarobeNote Added: 0020196
2009-09-29 14:11hgbotCheckin
2009-09-29 14:11hgbotNote Added: 0020458
2009-09-29 14:11hgbotStatusclosed => resolved
2009-09-29 14:11hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/c70b711ebf99bbf7f7e0842566299d7145156ada [^] => http://code.openbravo.com/erp/devel/pi-pageddatagrid/rev/c70b711ebf99bbf7f7e0842566299d7145156ada [^]
2009-10-06 11:16psarobeStatusresolved => closed
2009-10-06 11:16psarobeNote Added: 0020747

Notes
(0005968)
user71   
2005-06-01 00:00   
(edited on: 2008-06-12 09:43)
This bug was originally reported in SourceForge bug tracker and then migrated to Mantis.

You can see the original bug report in:
https://sourceforge.net/support/tracker.php?aid=1852662 [^]
(0002817)
rafaroda   
2008-01-10 11:22   
(edited on: 2008-06-12 09:23)
Logged In: YES
user_id=1892190
Originator: NO

Hi,

Could you please specify in detail the steps you follow to reproduce this bug?

Thank you very much,

Rafa Roda Palacios
Openbravo Team
(0002818)
villind   
2008-01-29 09:16   
(edited on: 2008-06-12 09:23)
Logged In: YES
user_id=61737
Originator: YES

Steps to reproduce:

 1. Click new "Business Partner"
 2. Enter details
 3. Open password dialog and give a new password
 4. Click save
 5. Log out
 6. Log in with the same user account as before (this fails as the password has been changed)
(0002819)
alostale   
2008-02-05 11:53   
(edited on: 2008-06-12 09:23)
Logged In: YES
user_id=1500722
Originator: NO

Which version are you working with?

Password is set for users (General Setup || Security || User) not for Business Partners. I tried to change the password to a user (with trunk version) and it didn't change the current user's one.
(0002820)
villind   
2008-02-05 14:06   
(edited on: 2008-06-12 09:23)
Logged In: YES
user_id=61737
Originator: YES

Steps to reproduce (on version 2.35mp1):

 1. Click new "User"
 2. Enter details
 3. Open password dialog and give a new password
 4. Click save
 5. Log out
 6. Log in with the same user account as before (this fails as the
password has been changed)

The same applied to trunk as well when this bug was reported.
(0002821)
cromero   
2008-02-05 18:23   
(edited on: 2008-06-12 09:23)
Logged In: YES
user_id=1500614
Originator: NO

The way to reproduce is not saving between steps 2 and 3.
An alert message is shown indicating that exist changes in the data that have not been saved (see attached file ScreenShot.png)
If you discard the message, you will edit the password of previous register shown before the new button was clicked (whatever it was).

Carlos Romero
Openbravo Team
File Added: ScreenShot.png
(0002822)
villind   
2008-02-05 19:30   
(edited on: 2008-06-12 09:23)
Logged In: YES
user_id=61737
Originator: YES

Yes, you need to cancel the warning to be able to enter the new password. But as passwords are usually entered while creating a new user some new Openbravo users fall into this trap and even re-install the application as the cannot login any more(see the IRC logs).

An easy way to fix this would be hiding the password entry when the user is not saved, but more optimal would be direct editing of password (and verify password) while creating a user.

The current implementation is not "wrong", but it is error prone.


(0002823)
alostale   
2008-02-15 18:24   
(edited on: 2008-06-12 09:23)
Logged In: YES
user_id=1500722
Originator: NO

The way it works now is showing an error message in case the user is not already saved.
(0008163)
plujan   
2008-07-03 19:33   
On r2.40beta is possible to repeat this behavior.
(0013267)
pjuvara   
2009-02-09 14:09   
Increasing priority as this is the oldest defect in the backlog.
(0013309)
shuehner   
2009-02-10 14:09   
In trunk the new record is auto-saved when you press on the 'Change password' button on a newly created (not yet saved user). Then the 'change password' dialog correctly changes the password of the just auto-saved user.

On disagreement with this explanation please do re-open the issue.
(0013327)
plujan   
2009-02-10 17:57   
Autosave function could be disabled and the behavior would be the same as in previous releases.
(0019818)
hgbot   
2009-09-11 19:57   
Repository: erp/devel/pi
Changeset: c70b711ebf99bbf7f7e0842566299d7145156ada
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Fri Sep 11 19:57:20 2009 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/c70b711ebf99bbf7f7e0842566299d7145156ada [^]

Fixed 2379: Hide change password field & change email server password field, if
autosave is disabled and record is not (yet) saved, as change password popup
only works with saved records.

---
M src-db/database/sourcedata/AD_FIELD.xml
---
(0019819)
hudsonbot   
2009-09-11 20:25   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-inc-pgsql/857/ [^]
------
(0019820)
hudsonbot   
2009-09-11 20:31   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-inc-oracle/750/ [^]
------
(0019821)
hudsonbot   
2009-09-11 20:50   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-full-pgsql/349/ [^]
------
(0019822)
hudsonbot   
2009-09-11 20:51   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-full-oracle/343/ [^]
------
(0019823)
hudsonbot   
2009-09-11 21:00   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-installation-test/92/ [^]
------
(0019832)
hudsonbot   
2009-09-11 21:17   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-pgsql-db-full-consistency-test/173/ [^]
------
(0019833)
hudsonbot   
2009-09-11 21:24   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-installation-test/93/ [^]
------
(0019848)
hudsonbot   
2009-09-11 23:15   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-integrity-test/137/ [^]
------
(0019852)
hudsonbot   
2009-09-12 09:53   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-pgsql-smoke-test/61/ [^]
------
(0019861)
hudsonbot   
2009-09-12 21:17   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-pgsql-db-full-consistency-test/174/ [^]
------
(0019862)
hudsonbot   
2009-09-13 09:54   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-pgsql-smoke-test/62/ [^]
------
(0019863)
hudsonbot   
2009-09-13 11:22   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-installation-test/94/ [^]
------
(0019874)
hudsonbot   
2009-09-13 21:17   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-pgsql-db-full-consistency-test/175/ [^]
------
(0019880)
hudsonbot   
2009-09-14 10:39   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-pgsql-smoke-test/63/ [^]
------
(0019897)
hudsonbot   
2009-09-14 11:51   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-integrity-test/138/ [^]
------
(0019911)
hudsonbot   
2009-09-14 14:54   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-installation-test/95/ [^]
------
(0019914)
hudsonbot   
2009-09-14 16:54   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-installation-test/96/ [^]
------
(0019922)
hudsonbot   
2009-09-14 18:37   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-pgsql-db-full-consistency-test/176/ [^]
------
(0019947)
hudsonbot   
2009-09-14 19:57   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-pgsql-db-full-consistency-test/177/ [^]
------
(0019971)
hudsonbot   
2009-09-14 21:07   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-installation-test/97/ [^]
------
(0019979)
hudsonbot   
2009-09-14 21:17   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-pgsql-db-full-consistency-test/178/ [^]
------
(0020001)
hudsonbot   
2009-09-14 23:15   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-integrity-test/139/ [^]
------
(0020013)
hudsonbot   
2009-09-15 01:09   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-installation-test/98/ [^]
------
(0020049)
hudsonbot   
2009-09-15 12:12   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-module-installation-test/99/ [^]
------
(0020062)
hudsonbot   
2009-09-15 12:35   
Fixed Issue 2379

------
Integrated in http://builds.openbravo.com/job/erp_devel_pi-pgsql-db-full-consistency-test/179/ [^]
------
(0020191)
sureshbabu   
2009-09-18 11:35   
I retested after i deactivated the auto functionality, after i create a user details when i click the password icon, warning message appears & i ignore the warning message by clicking OK and entered password

And i re login using the same user i can able to login in.

And also i find change password field & change email server password field are hidden fields.
(0020196)
psarobe   
2009-09-18 12:38   
Tested working fine
(0020458)
hgbot   
2009-09-29 14:11   
Repository: erp/devel/pi-pageddatagrid
Changeset: c70b711ebf99bbf7f7e0842566299d7145156ada
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Fri Sep 11 19:57:20 2009 +0200
URL: http://code.openbravo.com/erp/devel/pi-pageddatagrid/rev/c70b711ebf99bbf7f7e0842566299d7145156ada [^]

Fixed 2379: Hide change password field & change email server password field, if
autosave is disabled and record is not (yet) saved, as change password popup
only works with saved records.

---
M src-db/database/sourcedata/AD_FIELD.xml
---
(0020747)
psarobe   
2009-10-06 11:16   
Closed again because hudson has reopened without no reason