Henri,
Check the DTD version of the Faces example. The 1.3 should contain
cancellable only while 1.2 must have it as a <set-property> element.
Paul
Henri Yandell wrote:
The list of issues for 1.3.7 look good. I can't find your PGP key, so
I'm unable to check the .asc file.
The src in the -all zip happily packages into jars.
Installing the apps into a Tomcat 5.5, I get the following error for
struts-faces-example2-1.3.7:
SEVERE: Parse Error at line 146 column 37: Attribute "cancellable"
must be declared for element type "action".
org.xml.sax.SAXParseException: Attribute "cancellable" must be
declared for element type "action".
I can't recall if that's fine - does it mean that an app needs to move
from the <property> approach to doing cancellable?
Running the apps:
* Blank - Good.
* Cookbook - Problems on the Validator page:
When I enter the date it says "Date is not a date". I'm entering
27/2/1975 (and 27/02/1975, and 29/ and 28/). Looks like it is
mm/dd/yyyy and not dd/mm/yyyy as claimed. To prove the view source
works, from validation.xml via the webapp:
<var-value>MM/dd/yyyy</var-value>
The US Postal code mask allows more than 5 digits. Not sure if that's
bad.
* El-Example
Pressing Save on the Scalar Setters page causes an exception.
The Href link on the logic:redirect page leads to a 404
The last Logic Comparison tag test fails [result looks to be inverse
of the desired one]
* Examples
struts-bean:cookie results are incorrect for nulls. Shows blank.
MatchTag looks like it has a problem - its correct values and its test
results are from different semantic domains.
urlvalidator doesn't support http://user:[EMAIL PROTECTED]/ (I suspect
this is an enhancement req)
The rest of it seems good. I'll let you decide what's important in the
above. Assuming nothing seems like a blocker to you (ie: likely to be
related to the fixes), then I'm +1 on the release.
As to alpha/beta/GA, I'm new to that type of thing and not sure how to
define it as I don't run it in production.
Hen
On 2/25/07, Paul Benedict <[EMAIL PROTECTED]> wrote:
The Struts 1.3.7 test build is now available.
Release notes:
* http://struts.apache.org/1.x/userGuide/release-notes.html
Distribution:
* http://people.apache.org/builds/struts/1.3.7/
Maven 2 staging repository:
* http://people.apache.org/builds/struts/1.3.7/m2-staging-repository/
If you have had a chance to review the test build, please respond with
a vote on its quality:
[ ] Leave at test build
[ ] Alpha
[ ] Beta
[ ] General Availability (GA)
Everyone who has tested the build is invited to vote. Votes by PMC
members are considered binding. A vote passes if there are at least
three binding +1s and more +1s than -1s.
Please remember that a *binding* +1 for GA implies that you intend to
support the release by applying patches and responding to posts to the
user and dev lists.
The vote will remain open for at least 72 hours, longer upon request.
Thank you,
Paul Benedict
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
