DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=18394>.
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=18394

TilesRequestProcessor, downgrade log msg from error to info

           Summary: TilesRequestProcessor, downgrade log msg from error to
                    info
           Product: Struts
           Version: 1.1 RC1
          Platform: All
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Tiles framework
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


TilesRequestProcessor.initDefinitionsMapping() logs an error message if the 
definition factory was not found (line #131 in rev 1.17).

However, it is a legal situation to use the TilesRequestProcessor without a 
definition factory. Therefore, the error message should be downgraded to info 
(error messages irritate our operations people).

Specifically, our company has a legacy framework that wraps around Struts and 
also extends the (Tiles)RequestProcessor. Our legacy framework being a generic 
solution that is used by a number of our applications, we had to decide whether 
to extend the TilesRequestProcessor or the Struts RequestProcessor directly. We 
decided to extend the TilesRequestProcessor in order to allow our applications 
to use Tiles if they so desire. This means that all of our applications will 
use the TilesRequestProcessor, but not all of them will actually use Tiles. 
This works fine, only problem is it generates this meaningless error message in 
the logs.

Proposed fix: downgrade log message from error to info 
(TilesRequestProcessor.java v1.17, line 131)

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to