Re: [Spacewalk-devel] [PATCH] Pam setting on the user page not saving

2011-10-13 Thread Tomas Lestach
On 10/12/2011 05:22 PM, Johannes Renner wrote: On 10/12/2011 05:00 PM, Tomas Lestach wrote: Hello Johannes, yes, you're right. I just have one comment to your patch - I'd move updatePamAttribute() in SelfEditAction.java file into the updateDetails method. If you're ok with the change, I'd

[Spacewalk-devel] [PATCH] Pam setting on the user page not saving

2011-10-12 Thread Johannes Renner
Hello, I just fixed a bug in Spacewalk about the pam setting on the user page not saving, it can be reproduced like this: 1. Configure pam on the server (e.g. echo pam_auth_service = rhn-satellite /etc/rhn/rhn.conf) 2. Restart the tomcat 3. Log into spacewalk as an org_admin 4. In the page

Re: [Spacewalk-devel] [PATCH] Pam setting on the user page not saving

2011-10-12 Thread Johannes Renner
On 10/12/2011 05:00 PM, Tomas Lestach wrote: Hello Johannes, yes, you're right. I just have one comment to your patch - I'd move updatePamAttribute() in SelfEditAction.java file into the updateDetails method. If you're ok with the change, I'd commit it. Let me know. Yes, I was just