Continuum

Edit user info page errors

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.1-alpha-1
  • Fix Version/s: 1.1-alpha-1
  • Component/s: Web interface
  • Labels:
    None
  • Environment:
    continuu-acegi branch
  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

When clicking on "Edit user info" menu link:

The user name box is empty

Null pointer exception trying to save the user because in line 131
user = userManager.getUser( username );
the returned value is null

In fact this should call userManager.getMyUser() to prevent people playing with fields and impersonating another user

Issue Links

relates to

Bug - A problem which impairs or prevents the functions of the product. CONTINUUM-858 Move role add/edit/delete form to user group edit page

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Carlos Sanchez added a comment -

When deleting roles the deletions are not persisted. Seems a better option to have checkboxes and be able to save all permission modifications at once

Show
Carlos Sanchez added a comment - When deleting roles the deletions are not persisted. Seems a better option to have checkboxes and be able to save all permission modifications at once
Hide
Permalink
Carlos Sanchez added a comment -

when editing the password if there's an error all roles values dissapear

Show
Carlos Sanchez added a comment - when editing the password if there's an error all roles values dissapear
Hide
Permalink
Carlos Sanchez added a comment -

Roles needs to be moved to user group

Show
Carlos Sanchez added a comment - Roles needs to be moved to user group
Hide
Permalink
Lester Ecarma added a comment -

I can't replicate most of the issues being described here. Instead, I've found other issues that aren't exactly the same as the ones above.

  • An extra user with empty username and password is added by default. This causes an error when trying to edit/delete that user (i.e., NPE in EditUserAction as described above).
  • In 'Edit user info' page the roles of the current user is not being retreived, resulting in a blank roles list. The roles, however, are being retrieved correctly when accessed via Users > edit.
  • In Users > edit > Save, user is redirected to a blank page (i.e., user/saveAccount.action).
  • The Users menu item is hidden to a user with manageUser "ONLY" role. The weird thing is that it is visible when accompanied by other Administration roles (e.g., manageSchedule or manageConfiguration).
  • When in Users > edit page, the 'Edit user info' menu link shows the username of the user being edited.
Show
Lester Ecarma added a comment - I can't replicate most of the issues being described here. Instead, I've found other issues that aren't exactly the same as the ones above. An extra user with empty username and password is added by default. This causes an error when trying to edit/delete that user (i.e., NPE in EditUserAction as described above). In 'Edit user info' page the roles of the current user is not being retreived, resulting in a blank roles list. The roles, however, are being retrieved correctly when accessed via Users > edit. In Users > edit > Save, user is redirected to a blank page (i.e., user/saveAccount.action). The Users menu item is hidden to a user with manageUser "ONLY" role. The weird thing is that it is visible when accompanied by other Administration roles (e.g., manageSchedule or manageConfiguration). When in Users > edit page, the 'Edit user info' menu link shows the username of the user being edited.
Hide
Permalink
Carlos Sanchez added a comment -

Fixed the extra user creation in ConfigurationAction

Show
Carlos Sanchez added a comment - Fixed the extra user creation in ConfigurationAction
Hide
Permalink
Lester Ecarma added a comment -

All the other issues I posted above have already been fixed as well.

Show
Lester Ecarma added a comment - All the other issues I posted above have already been fixed as well.

People

  • Assignee:
    Carlos Sanchez
    Reporter:
    Carlos Sanchez
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: