Author: pledbrook
Date: 2006-08-05 07:28:42-0700
New Revision: 10239
Modified:
branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManagePermissions.java
branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManageRoles.java
branches/release/b21/xdocs/changes.xml
Log:
Applied the patch for SCB1773 submitted by Ronny Voelker - thanks!
Modified: branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManagePermissions.java
Url: http://scarab.tigris.org/source/browse/scarab/branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManagePermissions.java?view=diff&rev=10239&p1=branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManagePermissions.java&p2=branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManagePermissions.java&r1=10238&r2=10239
==============================================================================
--- branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManagePermissions.java (original)
+++ branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManagePermissions.java 2006-08-05 07:28:42-0700
@@ -61,6 +61,7 @@
// Scarab Stuff
import org.tigris.scarab.om.ScarabUser;
+import org.tigris.scarab.om.ScarabUserManager;
import org.tigris.scarab.tools.ScarabLocalizationTool;
import org.tigris.scarab.util.ScarabConstants;
import org.tigris.scarab.actions.base.RequireLoginFirstAction;
@@ -147,6 +148,8 @@
Permission permission = TurbineSecurity.getPermission(name);
TurbineSecurity.removePermission(permission);
+ ScarabUserManager.getMethodResult().clear();
+
ScarabLocalizationTool l10n = getLocalizationTool(context);
String msg = l10n.format("PermissionDeleted", name);
getScarabRequestTool(context).setConfirmMessage(msg);
Modified: branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManageRoles.java
Url: http://scarab.tigris.org/source/browse/scarab/branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManageRoles.java?view=diff&rev=10239&p1=branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManageRoles.java&p2=branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManageRoles.java&r1=10238&r2=10239
==============================================================================
--- branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManageRoles.java (original)
+++ branches/release/b21/src/java/org/tigris/scarab/actions/admin/ManageRoles.java 2006-08-05 07:28:42-0700
@@ -62,6 +62,7 @@
// Scarab Stuff
import org.tigris.scarab.om.ScarabUser;
+import org.tigris.scarab.om.ScarabUserManager;
import org.tigris.scarab.tools.ScarabLocalizationTool;
import org.tigris.scarab.util.ScarabConstants;
import org.tigris.scarab.actions.base.RequireLoginFirstAction;
@@ -204,6 +205,8 @@
role.revoke(permission);
}
}
+
+ ScarabUserManager.getMethodResult().clear();
}
/**
Modified: branches/release/b21/xdocs/changes.xml
Url: http://scarab.tigris.org/source/browse/scarab/branches/release/b21/xdocs/changes.xml?view=diff&rev=10239&p1=branches/release/b21/xdocs/changes.xml&p2=branches/release/b21/xdocs/changes.xml&r1=10238&r2=10239
==============================================================================
--- branches/release/b21/xdocs/changes.xml (original)
+++ branches/release/b21/xdocs/changes.xml 2006-08-05 07:28:42-0700
@@ -7,6 +7,10 @@
<body>
<release version="b21" date="in-development">
+ <action dev="pledbrook" type="fix" due-to="Ronny Voelker" fix="SCB1773">
+ Permission and role changes no longer require the container, e.g. tomcat,
+ to be restarted in order for them to take effect.
+ </action>
<action dev="jorgeuriarte" type="add" fix="SCB1407">
Added support for NTLM (Windows) login integration. Take a look at
<a href="howto/windowsdomain-howto.html">this howto</a>.
lmpx.com only provides a reader for public news (NNTP) servers. It is not
affiliated with the servers or forums shown here and is not responsible for
the content of articles, which is written by their respective authors.