Issue Details (XML | Word | Printable)

Key: SHALE-365
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Trivial Trivial
Assignee: Unassigned
Reporter: Ingo Dueppe
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Shale

faces-config.xml is not valid.

Created: 19/Dec/06 09:46 PM   Updated: 23/Jan/07 04:40 PM
Return to search
Component/s: Remoting
Affects Version/s: 1.0.4-SNAPSHOT
Fix Version/s: 1.0.4

Flags: Patch


 Description  « Hide
During the last change of faces-config the managed-bean-scope definition has been forgotten. So currently the faces-config.xml inside shale-remoting is not valid.

I am not sure what is intended with the LOGGER-Bean, but I assumed that it should be application-scope.

Regards
Ingo

Index: .
===================================================================
--- . (revision 488706)
+++ . (working copy)
@@ -43,9 +43,8 @@
   <managed-bean>
     <!-- Default logging adapter implementation -->
       <managed-bean-name>org.apache.shale.remoting.LOGGER</managed-bean-name>
- <managed-bean-class>
- org.apache.shale.remoting.logger.DefaultLogger
- </managed-bean-class>
+ <managed-bean-class>org.apache.shale.remoting.logger.DefaultLogger</managed-bean-class>
+ <managed-bean-scope>application</managed-bean-scope>
   </managed-bean>
 
 </faces-config>


 All   Comments   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Craig McClanahan added a comment - 19/Dec/06 09:59 PM
You're absolutely correct that I broke this, and that the intended scope is "application". I just checked in a fix, which will be in the 20061220 nightly build.