[#SHALE-320] Catch-all for bugs identified by static code analysis tools

Shale

Created: 27/Oct/06 12:07 AM Updated: 23/Jan/07 04:40 PM

Return to search

Component/s:

None

Affects Version/s:

None

Fix Version/s:

1.0.4

Description

« Hide

Differs from ~~SHALE-310~~ in that the changes are functional, not cosmetic.

All

Comments

Change History

Subversion Commits

Sort Order:

Repository	Revision	Date	User	Message
ASF	#468202	Fri Oct 27 00:17:03 UTC 2006	rahul	Validators only work if the validation methods are static. ~~SHALE-320~~
				Files Changed
				MODIFY /shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/CommonsValidator.java

Repository	Revision	Date	User	Message
ASF	#468609	Sat Oct 28 01:52:32 UTC 2006	rahul	Use JCL instead of j.u.l (identified by unused property warning) ~~SHALE-320~~
				Files Changed
				MODIFY /shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/CommonsValidator.java

Rahul Akolkar made changes - 11/Nov/06 10:32 PM

Field	Original Value	New Value
Assignee		Rahul Akolkar [ rahul@apache.org ]

[ Permlink | « Hide ]

Checking such reports is more of a continual process, but I've fixed the couple of issues I had in mind for 1.0.4 (see commits tab).

Rahul Akolkar made changes - 19/Nov/06 04:25 AM

Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

Repository	Revision	Date	User	Message
ASF	#481399	Fri Dec 01 21:20:51 UTC 2006	rahul	SVUIDs ~~SHALE-320~~
				Files Changed
				MODIFY /shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/tag/IntegerValidatorTag.java MODIFY /shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/tag/IntegerConverterTag.java

Repository	Revision	Date	User	Message
ASF	#481403	Fri Dec 01 21:27:54 UTC 2006	rahul	Need to refresh the TCCL, rather than assigning it to a field (as appears to be the intent). ~~SHALE-320~~
				Files Changed
				MODIFY /shale/framework/trunk/shale-core/src/main/java/org/apache/shale/util/Messages.java

Repository	Revision	Date	User	Message
ASF	#481404	Fri Dec 01 21:29:27 UTC 2006	rahul	Use the app map reference established a line ago. ~~SHALE-320~~
				Files Changed
				MODIFY /shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/CommonsValidator.java

Repository	Revision	Date	User	Message
ASF	#481412	Fri Dec 01 21:45:21 UTC 2006	rahul	Seems to be inconsequential. ~~SHALE-320~~
				Files Changed
				MODIFY /shale/framework/trunk/shale-validator/src/main/java/org/apache/shale/validator/faces/ValidatorTag.java

Rahul Akolkar made changes - 23/Jan/07 04:40 PM

Fix Version/s	1.0.4-SNAPSHOT [ 21740 ]
Fix Version/s		1.0.4 [ 21790 ]

Jeff Turner made changes - 09/Aug/07 07:15 AM

Workflow

Struts [ 38858 ]

Struts - editable closed status [ 41663 ]

Antonio Petrelli made changes - 08/Jan/09 08:56 AM

Workflow

Struts - editable closed status [ 41663 ]

Struts - editable closed status (temporary) [ 45903 ]

Antonio Petrelli made changes - 08/Jan/09 09:08 AM

Workflow

Struts - editable closed status (temporary) [ 45903 ]

Struts - editable closed status [ 52572 ]