cvs commit: jakarta-jetspeed/xdocs changes.xml

2003-01-21 Thread morciuch 
morciuch2003/01/21 13:50:33

  Modified:docs/site changes.html
   src/java/org/apache/jetspeed/services/security/registry
RegistryAccessController.java
   xdocschanges.xml
  Log:
  Eliminated one unnecessary registry lookup and log entry in the registry access 
controller (see Bugzilla bug# 16310).
  
  Revision  ChangesPath
  1.101 +3 -0  jakarta-jetspeed/docs/site/changes.html
  
  Index: changes.html
  ===
  RCS file: /home/cvs/jakarta-jetspeed/docs/site/changes.html,v
  retrieving revision 1.100
  retrieving revision 1.101
  diff -u -r1.100 -r1.101
  --- changes.html  21 Jan 2003 21:28:24 -  1.100
  +++ changes.html  21 Jan 2003 21:50:32 -  1.101
  @@ -133,6 +133,9 @@
   
   -->
   
  +  Fixed - Bug # 16310 - 2003/01/21 - Eliminating one unnecessary registry lookup 
and log entry in the registry access controller(MO)
  +
  +
 Fixed - Bug # 16309 - 2003/01/21 - Localized "no access to portlets" message in 
PortletWrapper (MO)
   
   
  
  
  
  1.5   +7 -2  
jakarta-jetspeed/src/java/org/apache/jetspeed/services/security/registry/RegistryAccessController.java
  
  Index: RegistryAccessController.java
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed/src/java/org/apache/jetspeed/services/security/registry/RegistryAccessController.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- RegistryAccessController.java 25 Aug 2002 19:45:07 -  1.4
  +++ RegistryAccessController.java 21 Jan 2003 21:50:33 -  1.5
  @@ -125,7 +125,12 @@
   }
   
   String portletName = portlet.getName();
  -RegistryEntry registryEntry = 
(RegistryEntry)Registry.getEntry(Registry.PORTLET, portletName);
  +RegistryEntry registryEntry = null;
  +// Don't query registry if portlet is a set
  +if (!(portlet instanceof PortletSet))
  +{
  +registryEntry = (RegistryEntry) Registry.getEntry(Registry.PORTLET, 
portletName);
  +}
   //portlet is not a portlet - probably a controller or control
   if (registryEntry==null) {
   PortletSet ps  = portlet.getPortletConfig().getPortletSet();
  
  
  
  1.118 +4 -1  jakarta-jetspeed/xdocs/changes.xml
  
  Index: changes.xml
  ===
  RCS file: /home/cvs/jakarta-jetspeed/xdocs/changes.xml,v
  retrieving revision 1.117
  retrieving revision 1.118
  diff -u -r1.117 -r1.118
  --- changes.xml   21 Jan 2003 21:28:24 -  1.117
  +++ changes.xml   21 Jan 2003 21:50:33 -  1.118
  @@ -23,6 +23,9 @@
   
   -->
   
  +  Fixed - Bug # 16310 - 2003/01/21 - Eliminating one unnecessary registry lookup 
and log entry in the registry access controller(MO)
  +
  +
 Fixed - Bug # 16309 - 2003/01/21 - Localized "no access to portlets" message in 
PortletWrapper (MO)
   
   
  
  
  

--
To unsubscribe, e-mail:   
For additional commands, e-mail:

cvs commit: jakarta-jetspeed/src/java/org/apache/jetspeed/portal/security/portlets PortletWrapper.java

2003-01-21 Thread morciuch 
morciuch2003/01/21 13:33:57

  Modified:src/java/org/apache/jetspeed/modules/localization
JetspeedLocalization_en.properties
   src/java/org/apache/jetspeed/portal/security/portlets
PortletWrapper.java
  Log:
  "No access to portlet" message is now localized - message correction (see Bugzilla 
bug# 16309).
  
  Revision  ChangesPath
  1.28  +2 -1  
jakarta-jetspeed/src/java/org/apache/jetspeed/modules/localization/JetspeedLocalization_en.properties
  
  Index: JetspeedLocalization_en.properties
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed/src/java/org/apache/jetspeed/modules/localization/JetspeedLocalization_en.properties,v
  retrieving revision 1.27
  retrieving revision 1.28
  diff -u -r1.27 -r1.28
  --- JetspeedLocalization_en.properties21 Jan 2003 20:43:35 -  1.27
  +++ JetspeedLocalization_en.properties21 Jan 2003 21:33:57 -  1.28
  @@ -16,6 +16,7 @@
   LOGINHELP_NOTICE=If you need additional assistance, please send email to the 
address below.
   
   SECURITY_NO_ACCESS=You do not have access to these portlets.
  +SECURITY_NO_ACCESS_TO_PORTLET=You do not have access to this portlet.
   
   PASSWORDREMINDER_EMAIL_HEADER=Hello
   PASSWORDREMINDER_EMAIL_BODY1=Your jetspeed account password is:
  
  
  
  1.19  +2 -2  
jakarta-jetspeed/src/java/org/apache/jetspeed/portal/security/portlets/PortletWrapper.java
  
  Index: PortletWrapper.java
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed/src/java/org/apache/jetspeed/portal/security/portlets/PortletWrapper.java,v
  retrieving revision 1.18
  retrieving revision 1.19
  diff -u -r1.18 -r1.19
  --- PortletWrapper.java   21 Jan 2003 21:28:24 -  1.18
  +++ PortletWrapper.java   21 Jan 2003 21:33:57 -  1.19
  @@ -153,7 +153,7 @@
   {
   PortletStats.logAccess(rundata, this, PortletStats.ACCESS_DENIED);
   }
  -return new 
JetspeedClearElement(Localization.getString("SECURITY_NO_ACCESS"));
  +return new 
JetspeedClearElement(Localization.getString("SECURITY_NO_ACCESS_TO_PORTLET"));
   }
   }
   
  
  
  

--
To unsubscribe, e-mail:   
For additional commands, e-mail:

cvs commit: jakarta-jetspeed/xdocs changes.xml

2003-01-21 Thread morciuch 
morciuch2003/01/21 13:28:24

  Modified:docs/site changes.html
   src/java/org/apache/jetspeed/portal/security/portlets
PortletWrapper.java
   xdocschanges.xml
  Log:
  "No access to portlets" message is now localized (see Bugzilla bug# 16309).
  
  Revision  ChangesPath
  1.100 +3 -0  jakarta-jetspeed/docs/site/changes.html
  
  Index: changes.html
  ===
  RCS file: /home/cvs/jakarta-jetspeed/docs/site/changes.html,v
  retrieving revision 1.99
  retrieving revision 1.100
  diff -u -r1.99 -r1.100
  --- changes.html  21 Jan 2003 20:43:35 -  1.99
  +++ changes.html  21 Jan 2003 21:28:24 -  1.100
  @@ -133,6 +133,9 @@
   
   -->
   
  +  Fixed - Bug # 16309 - 2003/01/21 - Localized "no access to portlets" message in 
PortletWrapper (MO)
  +
  +
 Fixed - Bug # 15414 - 2003/01/21 - Addressed problem with Customizer only 
allowing one Reference to be added (MO)
   
   
  
  
  
  1.18  +3 -2  
jakarta-jetspeed/src/java/org/apache/jetspeed/portal/security/portlets/PortletWrapper.java
  
  Index: PortletWrapper.java
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed/src/java/org/apache/jetspeed/portal/security/portlets/PortletWrapper.java,v
  retrieving revision 1.17
  retrieving revision 1.18
  diff -u -r1.17 -r1.18
  --- PortletWrapper.java   10 Jan 2003 23:06:04 -  1.17
  +++ PortletWrapper.java   21 Jan 2003 21:28:24 -  1.18
  @@ -72,6 +72,7 @@
   // Turbine imports
   import org.apache.turbine.util.Log;
   import org.apache.turbine.util.RunData;
  +import org.apache.turbine.services.localization.Localization;
   
   // ECS
   import org.apache.ecs.ConcreteElement;
  @@ -152,7 +153,7 @@
   {
   PortletStats.logAccess(rundata, this, PortletStats.ACCESS_DENIED);
   }
  -return new JetspeedClearElement("Sorry, you have no permission to see 
this portlet");
  +return new 
JetspeedClearElement(Localization.getString("SECURITY_NO_ACCESS"));
   }
   }
   
  
  
  
  1.117 +4 -1  jakarta-jetspeed/xdocs/changes.xml
  
  Index: changes.xml
  ===
  RCS file: /home/cvs/jakarta-jetspeed/xdocs/changes.xml,v
  retrieving revision 1.116
  retrieving revision 1.117
  diff -u -r1.116 -r1.117
  --- changes.xml   21 Jan 2003 20:43:36 -  1.116
  +++ changes.xml   21 Jan 2003 21:28:24 -  1.117
  @@ -23,6 +23,9 @@
   
   -->
   
  +  Fixed - Bug # 16309 - 2003/01/21 - Localized "no access to portlets" message in 
PortletWrapper (MO)
  +
  +
 Fixed - Bug # 15414 - 2003/01/21 - Addressed problem with Customizer only 
allowing one Reference to be added (MO)
   
   
  
  
  

--
To unsubscribe, e-mail:   
For additional commands, e-mail:

JSR-168 Pluto

2003-01-21 Thread Andrew C. Oliver 
Those Jetspeed and Cocoon Portal committers whom are interested should 
please add themselves as an initial committer 
http://nagoya.apache.org/wiki/apachewiki.cgi?PlutoProposal and 
participate in the currently scattered discussions on:

[EMAIL PROTECTED]
[EMAIL PROTECTED]
[EMAIL PROTECTED]

Thanks,

-Andy




--
To unsubscribe, e-mail:   
For additional commands, e-mail:

DO NOT REPLY [Bug 15414] - Customizer only allows one Reference to be added

2003-01-21 Thread bugzilla 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=15414

Customizer only allows one Reference to be added

[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|jetspeed-   |[EMAIL PROTECTED]
   |[EMAIL PROTECTED]  |



--- Additional Comments From [EMAIL PROTECTED]  2003-01-21 20:49 ---
The error message is currently set but not being displayed. We should reserve 
an area (possibly below navigation) to display any rundata messages.

--
To unsubscribe, e-mail:   
For additional commands, e-mail:

cvs commit: jakarta-jetspeed/xdocs changes.xml

2003-01-21 Thread morciuch 
morciuch2003/01/21 12:43:36

  Modified:docs/site changes.html
   src/java/org/apache/jetspeed/modules/actions/portlets
CustomizeSetAction.java
   src/java/org/apache/jetspeed/modules/localization
JetspeedLocalization_en.properties
   xdocschanges.xml
  Log:
  When attempting to add reference to a non-protected psml, a warning is now logged + 
rundata message is set (see Bugzilla bug# 15414).
  
  Although the message is set, it is currently not displayed anywhere. This should be 
addressed separately at a global level.
  
  Revision  ChangesPath
  1.99  +4 -1  jakarta-jetspeed/docs/site/changes.html
  
  Index: changes.html
  ===
  RCS file: /home/cvs/jakarta-jetspeed/docs/site/changes.html,v
  retrieving revision 1.98
  retrieving revision 1.99
  diff -u -r1.98 -r1.99
  --- changes.html  17 Jan 2003 21:30:37 -  1.98
  +++ changes.html  21 Jan 2003 20:43:35 -  1.99
  @@ -133,10 +133,13 @@
   
   -->
   
  +  Fixed - Bug # 15414 - 2003/01/21 - Addressed problem with Customizer only 
allowing one Reference to be added (MO)
  +
  +
 Fixed - Bug # 16187 - 2003/01/17 - Fixed "Error retrieving Portal Page" received 
after new user changes password (MO)
   
   
  -  Add - Bug # 15215 - 2003/01/17 - Added Jetspeed Questionnaire portlet (MO)
  +  Added - Bug # 15215 - 2003/01/17 - Added Jetspeed Questionnaire portlet (MO)
   
   
 Update - Bug # 16174 - 2003/01/16 - Enabled security ref selection in the 
customizers for user role (MO)
  
  
  
  1.38  +17 -3 
jakarta-jetspeed/src/java/org/apache/jetspeed/modules/actions/portlets/CustomizeSetAction.java
  
  Index: CustomizeSetAction.java
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed/src/java/org/apache/jetspeed/modules/actions/portlets/CustomizeSetAction.java,v
  retrieving revision 1.37
  retrieving revision 1.38
  diff -u -r1.37 -r1.38
  --- CustomizeSetAction.java   13 Jan 2003 17:24:35 -  1.37
  +++ CustomizeSetAction.java   21 Jan 2003 20:43:35 -  1.38
  @@ -105,7 +105,7 @@
   import org.apache.turbine.util.DynamicURI;
   import org.apache.turbine.util.Log;
   import org.apache.turbine.util.RunData;
  -
  +import org.apache.turbine.services.localization.Localization;
   
   
   // Velocity Stuff
  @@ -121,7 +121,7 @@
   import java.util.Iterator;
   import java.util.Collections;
   import java.util.Comparator;
  -
  +import java.text.MessageFormat;
   
   /**
* This action implements the default portletset behavior customizer 
  @@ -717,6 +717,20 @@
   ref.setPath(refNames[i]);  
   ref.setSecurityRef(sref);
   portlets.addReference(ref);
  +}
  +else
  +{
  +String tmpl = 
Localization.getString("CUSTOMIZER_ADD_REF_ERROR");
  +Object[] args = {
  +refNames[i]
  +};
  +String message = MessageFormat.format(tmpl, args).toString();
  +
  +rundata.addMessage(message.concat(""));
  +if (Log.getLogger().isWarnEnabled())
  +{
  +Log.warn(message);
  +}
   }
   }
   }
  
  
  
  1.27  +2 -1  
jakarta-jetspeed/src/java/org/apache/jetspeed/modules/localization/JetspeedLocalization_en.properties
  
  Index: JetspeedLocalization_en.properties
  ===
  RCS file: 
/home/cvs/jakarta-jetspeed/src/java/org/apache/jetspeed/modules/localization/JetspeedLocalization_en.properties,v
  retrieving revision 1.26
  retrieving revision 1.27
  diff -u -r1.26 -r1.27
  --- JetspeedLocalization_en.properties8 Dec 2002 19:33:52 -   1.26
  +++ JetspeedLocalization_en.properties21 Jan 2003 20:43:35 -  1.27
  @@ -180,6 +180,7 @@
   CUSTOMIZER_TITLE=Title
   CUSTOMIZER_TYPE=Type
   CUSTOMIZER_UPDATE=Update
  +CUSTOMIZER_ADD_REF_ERROR=Psml [{0}] must have a security ref in order to be 
referenced
   
   PASSWORDFORM_TITLE=Change your account password
   PASSWORDFORM_USERNAMEMSG=Username:
  
  
  
  1.116 +5 -2  jakarta-jetspeed/xdocs/changes.xml
  
  Index: changes.xml
  ===
  RCS file: /home/cvs/jakarta-jetspeed/xdocs/changes.xml,v
  retrieving revision 1.115
  retrieving revision 1.116
  diff -u -r1.115 -r1.116
  --- changes.xml   17 Jan 2003 21:30:37 -  1.115
  +++ changes.xml   21 Jan 2003 20:43:36 -  1.116
  @@ -23,10 +23,13 @@
   
   -->
   
  +  Fixed - Bug # 15414 - 2003/01/21 - Addressed problem with Customizer only 
allowing one Reference to

DO NOT REPLY [Bug 15414] - Customizer only allows one Reference to be added

2003-01-21 Thread bugzilla 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=15414

Customizer only allows one Reference to be added

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Target Milestone|1.4b2   |1.4b4
Version|1.4b4-dev /CVS  |1.4b2



--- Additional Comments From [EMAIL PROTECTED]  2003-01-21 18:21 ---
There's logic in the customizer which prevents adding references that are not 
protected with a root security ref. No error is reported to the user and no 
warning is logged when attempting to add unprotected reference - it is simply 
ignored.

I'm not totally clear what is the reasoning behind this rule but I propose to 
at least log a warning in the log. Displaying a message to the user would be 
nice too.

--
To unsubscribe, e-mail:   
For additional commands, e-mail:

JetspeedLocalization_it 1.4b3

2003-01-21 Thread Massimiliano Dessì 
jetspeedLocalization_it.properties for jetspeed1.4b3

Massimiliano Dessì
Cagliari (Italy)

__
Yahoo! Cellulari: loghi, suonerie, picture message per il tuo telefonino
http://it.yahoo.com/mail_it/foot/?http://it.mobile.yahoo.com/index2002.html

--
To unsubscribe, e-mail:   
For additional commands, e-mail: