Bug 8

Summary:	I could add an attribute, but couldn't remove it
Product:	Websites	Reporter:	Samuel Verschelde <stormi-mageia>
Component:	identity.mageia.org	Assignee:	Romain d'Alverny <rdalverny-OLD>
Status:	RESOLVED FIXED	QA Contact:
Severity:	enhancement
Priority:	Normal	CC:	bgmilne, dmorganec
Version:	trunk
Target Milestone:	---
Hardware:	i586
OS:	Linux
Whiteboard:
Source RPM:		CVE:
Status comment:

Description Samuel Verschelde 2011-02-11 22:58:23 CET

In my identity.mageia.org page, as I'm a standard user, I tried to play with the "Add a attribute" (which should be "Add an attribute") form, and added a "carLicense" attribute.

But now I can't remove it anymore. Clicking on "delete" gives the following error : "Please come back later" (which is the most useless error message I have ever seen, but I guess it's catalyst's default message :))

Suggestion : just remove that "Add an attribute" thing for now ? (I still don't know how I should use it).

D Morgan 2011-02-12 23:31:56 CET

Priority: --- => Normal
CC: (none) => dmorganec

Comment 1 Buchan Milne 2011-02-16 14:40:31 CET

"Please come back later" is the Catalyst page that you get if there is an unhandled exception when not running in debug mode.

In trunk (https://identity-trunk.mageia.org) I have added error pages to fix this.

Additionally, I have made the /user page correctly respect the editable_attrs configuration item, so users will only be shown attributes we want them to be able to edit (which should match the ACLs in LDAP, and only be attributes we want to show to users).

I added a carLicense to my user, and it was still shown in the existing attributes, and I could delete it (as this user), but I could no longer add it. I believe this is (more or less) the correct behaviour.

If you can't delete the carLicense attribute, and you would like it deleted, I will delete it manually.

I am hoping to merge trunk (https://identity-trunk.mageia.org) to live (https://identity.mageia.org) this week, but you can use trunk too ...

Status: NEW => RESOLVED
CC: (none) => bgmilne
Resolution: (none) => FIXED

Comment 2 Samuel Verschelde 2011-02-16 15:15:50 CET

Thanks, I deleted it.

Comment 3 Buchan Milne 2011-02-16 23:22:25 CET

trunk has been merged to live, so all issues mentioned as fixed in trunk are now fixed on live.