[jira] [Commented] (TAP5-1474) [GSoC] add out-of-the-box protection against cross-site request forgery (CSRF)

2013-09-03 Thread JIRA

[ 
https://issues.apache.org/jira/browse/TAP5-1474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13756431#comment-13756431
 ] 

Christian Köberl commented on TAP5-1474:


Is there any plan for including the finished module 
(http://code.google.com/p/gsoc2011-csrf-protection/) in Tapestry?

 [GSoC] add out-of-the-box protection against cross-site request forgery (CSRF)
 --

 Key: TAP5-1474
 URL: https://issues.apache.org/jira/browse/TAP5-1474
 Project: Tapestry 5
  Issue Type: New Feature
  Components: tapestry-core
Affects Versions: 5.2
Reporter: Ulrich Stärk
  Labels: gsoc2011

 There are several approaches to protect against CSRF. A student working on 
 this task will evaluate the possible solutions, discuss with the community 
 which to implement and implement and test the chosen approach.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Reopened] (TAP5-1973) :443 added to URLs when using the Link.toAbsoluteURI(true)

2013-09-03 Thread Kalle Korhonen (JIRA)

 [ 
https://issues.apache.org/jira/browse/TAP5-1973?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Kalle Korhonen reopened TAP5-1973:
--

  Assignee: Kalle Korhonen  (was: Howard M. Lewis Ship)

 :443 added to URLs when using the Link.toAbsoluteURI(true)
 --

 Key: TAP5-1973
 URL: https://issues.apache.org/jira/browse/TAP5-1973
 Project: Tapestry 5
  Issue Type: Bug
  Components: tapestry-core
Affects Versions: 5.3.4, 5.4
Reporter: Howard M. Lewis Ship
Assignee: Kalle Korhonen
  Labels: fixed-in-5.4-js-rewrite
 Fix For: 5.3.5, 5.4


 An error in the code means that when secure is true and the port is set to 
 443, then :443 is appended:
 int port = secure ? secureHostPort : hostPort;
 String portSuffix = ;
 if (port = 0) { 
 port = request.getServerPort();
 int schemeDefaultPort = request.isSecure() ? 443 : 80;
 portSuffix = port == schemeDefaultPort ?  : : + port;
 }
 else if (secure  port != 443) portSuffix = : + port;
 else if (port != 80) portSuffix = : + port;
 
 String hostname = .equals(this.hostname) ? request.getServerName() 
 : this.hostname.startsWith($) ? System.getenv(this.hostname.substring(1)) : 
 this.hostname;
 
 return String.format(%s://%s%s, secure ? https : http, 
 hostname, portSuffix);
 secure == true  port != 443 is false, so
 port != 80 is evaluated, it true, so
 :443 is appended.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Assigned] (TAP5-2165) Resource name case check breaks applications on windows

2013-09-03 Thread Howard M. Lewis Ship (JIRA)

 [ 
https://issues.apache.org/jira/browse/TAP5-2165?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Howard M. Lewis Ship reassigned TAP5-2165:
--

Assignee: Howard M. Lewis Ship

 Resource name case check breaks applications on windows
 ---

 Key: TAP5-2165
 URL: https://issues.apache.org/jira/browse/TAP5-2165
 Project: Tapestry 5
  Issue Type: Bug
  Components: tapestry-core
Affects Versions: 5.4
Reporter: Ville Virtanen
Assignee: Howard M. Lewis Ship
Priority: Blocker

 The fix made in TAP5-1007 broke apps when developed in windows probably due 
 to hard coded path separator. Discussion in mailing list: 
 http://apache-tapestry-mailing-list-archives.1045711.n5.nabble.com/5-4-alpha-15-resource-exceptions-td5723122.html
  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


git commit: TAP5-2165: Resource name case check breaks applications on windows

2013-09-03 Thread hlship
Updated Branches:
  refs/heads/master 2f0612bb5 - 0d784d945


TAP5-2165: Resource name case check breaks applications on windows


Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/0d784d94
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/0d784d94
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/0d784d94

Branch: refs/heads/master
Commit: 0d784d94547cbcb06a53b949c9e293a1922a7aa5
Parents: 2f0612b
Author: Howard M. Lewis Ship hls...@apache.org
Authored: Tue Sep 3 16:05:09 2013 -0700
Committer: Howard M. Lewis Ship hls...@apache.org
Committed: Tue Sep 3 16:05:18 2013 -0700

--
 .../apache/tapestry5/ioc/internal/util/AbstractResource.java| 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/0d784d94/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/util/AbstractResource.java
--
diff --git 
a/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/util/AbstractResource.java
 
b/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/util/AbstractResource.java
index 23773cd..7020cb9 100644
--- 
a/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/util/AbstractResource.java
+++ 
b/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/util/AbstractResource.java
@@ -318,7 +318,10 @@ public abstract class AbstractResource extends LockSupport 
implements Resource
 
 try
 {
-expectedFileName = extractFile(file.getCanonicalPath());
+// On Windows, the canonical path uses backslash ('\') for the 
separator; an easy hack
+// is to convert the platform file separator to match sane 
operating systems (which use a foward slash).
+String sep = System.getProperty(file.separator);
+expectedFileName = 
extractFile(file.getCanonicalPath().replace(sep, /));
 } catch (IOException e)
 {
 return;



[jira] [Closed] (TAP5-2165) Resource name case check breaks applications on windows

2013-09-03 Thread Howard M. Lewis Ship (JIRA)

 [ 
https://issues.apache.org/jira/browse/TAP5-2165?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Howard M. Lewis Ship closed TAP5-2165.
--

   Resolution: Fixed
Fix Version/s: 5.4

 Resource name case check breaks applications on windows
 ---

 Key: TAP5-2165
 URL: https://issues.apache.org/jira/browse/TAP5-2165
 Project: Tapestry 5
  Issue Type: Bug
  Components: tapestry-core
Affects Versions: 5.4
Reporter: Ville Virtanen
Assignee: Howard M. Lewis Ship
Priority: Blocker
 Fix For: 5.4


 The fix made in TAP5-1007 broke apps when developed in windows probably due 
 to hard coded path separator. Discussion in mailing list: 
 http://apache-tapestry-mailing-list-archives.1045711.n5.nabble.com/5-4-alpha-15-resource-exceptions-td5723122.html
  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (TAP5-2165) Resource name case check breaks applications on windows

2013-09-03 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/TAP5-2165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13757270#comment-13757270
 ] 

Hudson commented on TAP5-2165:
--

SUCCESS: Integrated in tapestry-trunk-freestyle #1144 (See 
[https://builds.apache.org/job/tapestry-trunk-freestyle/1144/])
TAP5-2165: Resource name case check breaks applications on windows (hlship: rev 
0d784d94547cbcb06a53b949c9e293a1922a7aa5)
* 
tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/util/AbstractResource.java


 Resource name case check breaks applications on windows
 ---

 Key: TAP5-2165
 URL: https://issues.apache.org/jira/browse/TAP5-2165
 Project: Tapestry 5
  Issue Type: Bug
  Components: tapestry-core
Affects Versions: 5.4
Reporter: Ville Virtanen
Assignee: Howard M. Lewis Ship
Priority: Blocker
 Fix For: 5.4


 The fix made in TAP5-1007 broke apps when developed in windows probably due 
 to hard coded path separator. Discussion in mailing list: 
 http://apache-tapestry-mailing-list-archives.1045711.n5.nabble.com/5-4-alpha-15-resource-exceptions-td5723122.html
  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[1/2] git commit: Change Label (back) to rendering class=control-label

2013-09-03 Thread hlship
Updated Branches:
  refs/heads/master 0d784d945 - a586a1af6


Change Label (back) to rendering class=control-label


Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/9df0250b
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/9df0250b
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/9df0250b

Branch: refs/heads/master
Commit: 9df0250b46fcacbcc566517a0d281273f9271238
Parents: 0d784d9
Author: Howard M. Lewis Ship hls...@apache.org
Authored: Tue Sep 3 16:26:49 2013 -0700
Committer: Howard M. Lewis Ship hls...@apache.org
Committed: Tue Sep 3 16:26:49 2013 -0700

--
 54_RELEASE_NOTES.txt | 11 +++
 .../org/apache/tapestry5/corelib/components/Label.java   |  2 +-
 .../apache/tapestry5/integration/app1/MiscTests.groovy   |  4 ++--
 .../org/apache/tapestry5/integration/app1/FormTests.java |  6 --
 4 files changed, 18 insertions(+), 5 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/9df0250b/54_RELEASE_NOTES.txt
--
diff --git a/54_RELEASE_NOTES.txt b/54_RELEASE_NOTES.txt
index faad914..1840d92 100644
--- a/54_RELEASE_NOTES.txt
+++ b/54_RELEASE_NOTES.txt
@@ -196,6 +196,17 @@ http://twitter.github.com/bootstrap/base-css.html#forms
 
 You may also use the new FormGroup mixin for most fields.
 
+## Form element components
+
+TextField, PasswordField, TextArea, and Select now render the CSS class 
attribute form-control; you may add additional
+CSS class names with the `class` informal parameter.  Generally, you will want 
to add Bootstrap `col-md-x` class names
+to control the size of the element (otherwise it will stretch to 100% of the 
available width).
+
+## LabelComponent
+
+The Label component now renders the CSS class attribute as control-label; 
additional CSS class names may be provided
+using the `class` informal parameter.
+
 ## BeanEditor / BeanEditForm
 
 The property edit blocks contributed to the BeanBlockSource service should 
expect to be nested inside a

http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/9df0250b/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/Label.java
--
diff --git 
a/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/Label.java
 
b/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/Label.java
index 7bb9f4a..b030929 100644
--- 
a/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/Label.java
+++ 
b/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/Label.java
@@ -64,7 +64,7 @@ public class Label
 {
 decorator.beforeLabel(field);
 
-labelElement = writer.element(label);
+labelElement = writer.element(label, class, control-label);
 
 resources.renderInformalParameters(writer);
 

http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/9df0250b/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/MiscTests.groovy
--
diff --git 
a/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/MiscTests.groovy
 
b/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/MiscTests.groovy
index f0a5c3d..b2bde4e 100644
--- 
a/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/MiscTests.groovy
+++ 
b/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/MiscTests.groovy
@@ -40,8 +40,8 @@ class MiscTests extends TapestryCoreTestCase {
 {
 openLinks Override Label Class Demo
 
-assertSourcePresent label for=\firstName\First Name/label,
-label for=\lastName\ 
class=\dummyClassName\Last Name/label
+assertSourcePresent label for=\firstName\ 
class=\control-label\First Name/label,
+label for=\lastName\ class=\control-label 
dummyClassName\Last Name/label
 
 }
 

http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/9df0250b/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java
--
diff --git 
a/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java
 
b/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java
index 4c551b6..9e4ce92 100644
--- 
a/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java
+++ 
b/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/FormTests.java
@@ -16,6 +16,7 @@ package org.apache.tapestry5.integration.app1;
 
 import org.apache.tapestry5.corelib.components.Form;
 import 

[2/2] git commit: Add class=form-control to the KaptchaField

2013-09-03 Thread hlship
Add class=form-control to the KaptchaField


Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/a586a1af
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/a586a1af
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/a586a1af

Branch: refs/heads/master
Commit: a586a1af6df28fcf9e2a8567797039c187c332ea
Parents: 9df0250
Author: Howard M. Lewis Ship hls...@apache.org
Authored: Tue Sep 3 17:07:07 2013 -0700
Committer: Howard M. Lewis Ship hls...@apache.org
Committed: Tue Sep 3 17:07:07 2013 -0700

--
 .../kaptcha/components/KaptchaField.java|  7 ++-
 .../java/kaptcha/demo/components/Layout.java|  5 +-
 .../kaptcha/demo/components/Layout.tml  | 24 --
 .../test/resources/kaptcha/demo/pages/Index.tml | 12 ++---
 .../kaptcha/demo/pages/KaptchaDemo.tml  | 50 +---
 5 files changed, 44 insertions(+), 54 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/a586a1af/tapestry-kaptcha/src/main/java/org/apache/tapestry5/kaptcha/components/KaptchaField.java
--
diff --git 
a/tapestry-kaptcha/src/main/java/org/apache/tapestry5/kaptcha/components/KaptchaField.java
 
b/tapestry-kaptcha/src/main/java/org/apache/tapestry5/kaptcha/components/KaptchaField.java
index a559aba..880aaf8 100644
--- 
a/tapestry-kaptcha/src/main/java/org/apache/tapestry5/kaptcha/components/KaptchaField.java
+++ 
b/tapestry-kaptcha/src/main/java/org/apache/tapestry5/kaptcha/components/KaptchaField.java
@@ -1,4 +1,4 @@
-// Copyright 2011, 2012 The Apache Software Foundation
+// Copyright 2011-2013 The Apache Software Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the License);
 // you may not use this file except in compliance with the License.
@@ -29,7 +29,8 @@ import org.apache.tapestry5.services.FieldValidatorSource;
 
 /**
  * Field paired with a {@link KaptchaImage} to ensure that the user has 
provided
- * the correct value.
+ * the correct value. Renders a class attribute of form-control, which can 
be extended
+ * via the class informal attribute.
  *
  * @since 5.3
  * @tapestrydoc
@@ -93,6 +94,8 @@ public class KaptchaField extends AbstractField
 
 id, getClientId(),
 
+class, form-control,
+
 name, getControlName(),
 
 value, visible ? validationTracker.getInput(this) : );

http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/a586a1af/tapestry-kaptcha/src/test/java/kaptcha/demo/components/Layout.java
--
diff --git a/tapestry-kaptcha/src/test/java/kaptcha/demo/components/Layout.java 
b/tapestry-kaptcha/src/test/java/kaptcha/demo/components/Layout.java
index f15a5a8..388df77 100644
--- a/tapestry-kaptcha/src/test/java/kaptcha/demo/components/Layout.java
+++ b/tapestry-kaptcha/src/test/java/kaptcha/demo/components/Layout.java
@@ -1,4 +1,4 @@
-// Copyright 2011 The Apache Software Foundation
+// Copyright 2011-2013 The Apache Software Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the License);
 // you may not use this file except in compliance with the License.
@@ -14,6 +14,9 @@
 
 package kaptcha.demo.components;
 
+import org.apache.tapestry5.annotations.Import;
+
+@Import(stack = core)
 public class Layout
 {
 

http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/a586a1af/tapestry-kaptcha/src/test/resources/kaptcha/demo/components/Layout.tml
--
diff --git 
a/tapestry-kaptcha/src/test/resources/kaptcha/demo/components/Layout.tml 
b/tapestry-kaptcha/src/test/resources/kaptcha/demo/components/Layout.tml
index 8156677..35e1143 100644
--- a/tapestry-kaptcha/src/test/resources/kaptcha/demo/components/Layout.tml
+++ b/tapestry-kaptcha/src/test/resources/kaptcha/demo/components/Layout.tml
@@ -1,23 +1,17 @@
-html xmlns=http://www.w3.org/1999/xhtml; 
xmlns:t=http://tapestry.apache.org/schema/tapestry_5_3.xsd; 
xmlns:p=tapestry:parameter
-  head
+html xmlns=http://www.w3.org/1999/xhtml; 
xmlns:t=http://tapestry.apache.org/schema/tapestry_5_3.xsd;
+head
 title${componentResources.pageName}/title
-  /head
+/head
+body
+div class=container
 
-  body
 h1${componentResources.pageName}/h1
 
 t:body/
 
-hr/
+t:pagelink class=btn btn-default page=indexindex/t:pagelink
 
-p
-  [
-  t:pagelink page=indexindex/t:pagelink
-  ] |
-  [
-  t:pagelink page=${componentResources.pageName}refresh/t:pagelink
-  ]
-/p
-
-  /body
+t:pagelink class=btn btn-default 
page=${componentResources.pageName}refresh/t:pagelink
+/div
+/body
 /html
\ No newline at end of file


git commit: Simplify layout

2013-09-03 Thread hlship
Updated Branches:
  refs/heads/master a586a1af6 - 3ae1187eb


Simplify layout


Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/3ae1187e
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/3ae1187e
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/3ae1187e

Branch: refs/heads/master
Commit: 3ae1187ebc9428c5fa5de030320d31596867de41
Parents: a586a1a
Author: Howard M. Lewis Ship hls...@apache.org
Authored: Tue Sep 3 17:11:05 2013 -0700
Committer: Howard M. Lewis Ship hls...@apache.org
Committed: Tue Sep 3 17:11:05 2013 -0700

--
 .../kaptcha/demo/pages/KaptchaDemo.tml  | 23 +++-
 1 file changed, 8 insertions(+), 15 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/3ae1187e/tapestry-kaptcha/src/test/resources/kaptcha/demo/pages/KaptchaDemo.tml
--
diff --git 
a/tapestry-kaptcha/src/test/resources/kaptcha/demo/pages/KaptchaDemo.tml 
b/tapestry-kaptcha/src/test/resources/kaptcha/demo/pages/KaptchaDemo.tml
index 228eb0f..5f30d5f 100644
--- a/tapestry-kaptcha/src/test/resources/kaptcha/demo/pages/KaptchaDemo.tml
+++ b/tapestry-kaptcha/src/test/resources/kaptcha/demo/pages/KaptchaDemo.tml
@@ -3,23 +3,16 @@
 p id=message${message}/p
 
 
-t:form t:id=form
-t:errors/
-
-div class=form-group
+t:form t:id=form
+t:errors/
 
+div class=form-group
 t:kaptchaimage t:id=ki/
-/div
-
-div class=form-group
-
-t:label for=kf class=col-2/
-div class=col-md-2
-t:kaptchafield label=Security Check t:id=kf 
image=ki/
-/div
-/div
+t:label for=kf/
+t:kaptchafield label=Security Check t:id=kf image=ki/
+/div
 
-input class=btn btn-primary type=submit value=Go/
-/t:form
+input class=btn btn-primary type=submit value=Go/
+/t:form
 
 /t:layout
\ No newline at end of file



[jira] [Updated] (TAP5-2163) Adapt DateField style to Bootstrap 3

2013-09-03 Thread Howard M. Lewis Ship (JIRA)

 [ 
https://issues.apache.org/jira/browse/TAP5-2163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Howard M. Lewis Ship updated TAP5-2163:
---

Summary: Adapt DateField style to Bootstrap 3  (was: Adapt DateField style 
to Bootstrap(3))

 Adapt DateField style to Bootstrap 3
 

 Key: TAP5-2163
 URL: https://issues.apache.org/jira/browse/TAP5-2163
 Project: Tapestry 5
  Issue Type: Bug
  Components: tapestry-core
Affects Versions: 5.4
Reporter: Jochen Kemnade
Assignee: Howard M. Lewis Ship
  Labels: bootstrap, css, javascript, patch, style
 Attachments: 0001-style-changes-to-adapt-to-Bootstrap-3.patch


 The DateField component's popup should be styled using Bootstrap 3 mechanisms.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


git commit: TAP5-2163: Adapt DateField style to Bootstrap 3

2013-09-03 Thread hlship
Updated Branches:
  refs/heads/master 3ae1187eb - 2cef4982e


TAP5-2163: Adapt DateField style to Bootstrap 3


Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/2cef4982
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/2cef4982
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/2cef4982

Branch: refs/heads/master
Commit: 2cef4982e017acaf3e6e0fc774db5fc19e49ca03
Parents: 3ae1187
Author: Howard M. Lewis Ship hls...@apache.org
Authored: Tue Sep 3 17:23:44 2013 -0700
Committer: Howard M. Lewis Ship hls...@apache.org
Committed: Tue Sep 3 17:23:44 2013 -0700

--
 .../META-INF/modules/t5/core/datefield.coffee   |  4 +--
 .../tapestry5/corelib/components/DateField.java |  2 +-
 .../tapestry5/datepicker_106/css/datepicker.css | 36 ++--
 .../tapestry5/datepicker_106/js/datepicker.js   | 16 ++---
 .../META-INF/assets/tapestry5/tapestry.css  |  2 +-
 5 files changed, 19 insertions(+), 41 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/2cef4982/tapestry-core/src/main/coffeescript/META-INF/modules/t5/core/datefield.coffee
--
diff --git 
a/tapestry-core/src/main/coffeescript/META-INF/modules/t5/core/datefield.coffee 
b/tapestry-core/src/main/coffeescript/META-INF/modules/t5/core/datefield.coffee
index a1b1bfd..20a9782 100644
--- 
a/tapestry-core/src/main/coffeescript/META-INF/modules/t5/core/datefield.coffee
+++ 
b/tapestry-core/src/main/coffeescript/META-INF/modules/t5/core/datefield.coffee
@@ -114,7 +114,7 @@ define [./dom, ./events, ./messages, ./ajax, 
underscore, ./fields],
   createPopup: -
 @datePicker = new DatePicker()
 @datePicker.setFirstWeekDay datePickerFirstDay
-@popup = dom.create(div, { class: datefield-popup}).append 
@datePicker.create()
+@popup = dom.create(div, { class: datefield-popup well}).append 
@datePicker.create()
 @container.insertAfter @popup
 
 @datePicker.onselect = _.bind @onSelect, this
@@ -160,4 +160,4 @@ define [./dom, ./events, ./messages, ./ajax, 
underscore, ./fields],
 dom.onDocument events.zone.didUpdate, - scan this
 
 # Exports nothing.
-return null
\ No newline at end of file
+return null

http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/2cef4982/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/DateField.java
--
diff --git 
a/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/DateField.java
 
b/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/DateField.java
index 99676de..96db67d 100644
--- 
a/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/DateField.java
+++ 
b/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/DateField.java
@@ -246,7 +246,7 @@ public class DateField extends AbstractField
 
 writer.element(button,
 type, button,
-class, btn,
+class, btn btn-default,
 alt, [Show]);
 
 writer.element(span, class, glyphicon glyphicon-calendar);

http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/2cef4982/tapestry-core/src/main/resources/META-INF/assets/tapestry5/datepicker_106/css/datepicker.css
--
diff --git 
a/tapestry-core/src/main/resources/META-INF/assets/tapestry5/datepicker_106/css/datepicker.css
 
b/tapestry-core/src/main/resources/META-INF/assets/tapestry5/datepicker_106/css/datepicker.css
index fb4cdaf..46794b8 100644
--- 
a/tapestry-core/src/main/resources/META-INF/assets/tapestry5/datepicker_106/css/datepicker.css
+++ 
b/tapestry-core/src/main/resources/META-INF/assets/tapestry5/datepicker_106/css/datepicker.css
@@ -1,8 +1,5 @@
 .datePicker {
-border: 1px solid WindowText;
-background: Window;
-width: 170px;
-padding: 0px;
+width: 180px;
 cursor: default;
 -moz-user-focus: normal;
 }
@@ -11,7 +8,6 @@
 font: smallcaption;
 font: small-caption;
 text-align: center;
-color: WindowText;
 cursor: default;
 font-weight: normal !important;
 -moz-user-select: none;
@@ -23,7 +19,6 @@
 }
 
 .datePicker .header {
-background: Window;
 padding: 3px;
 border-bottom: 1px solid WindowText;
 }
@@ -90,35 +85,10 @@
 width: 100%;
 }
 
-.datePicker button {
-border-width: 1px;
-font: Caption;
-font-weight: normal !important;
-display: block;
-}
-
-.datePicker button.btn {
-font-size: x-small;
-}
-
-.datePicker .previousButton {
-background: buttonface url(../images/arrow.left.png) no-repeat center 
center;
-}
-
-.datePicker .nextButton {
-background: buttonface 

[jira] [Closed] (TAP5-2163) Adapt DateField style to Bootstrap 3

2013-09-03 Thread Howard M. Lewis Ship (JIRA)

 [ 
https://issues.apache.org/jira/browse/TAP5-2163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Howard M. Lewis Ship closed TAP5-2163.
--

   Resolution: Fixed
Fix Version/s: 5.4

 Adapt DateField style to Bootstrap 3
 

 Key: TAP5-2163
 URL: https://issues.apache.org/jira/browse/TAP5-2163
 Project: Tapestry 5
  Issue Type: Bug
  Components: tapestry-core
Affects Versions: 5.4
Reporter: Jochen Kemnade
Assignee: Howard M. Lewis Ship
  Labels: bootstrap, css, javascript, patch, style
 Fix For: 5.4

 Attachments: 0001-style-changes-to-adapt-to-Bootstrap-3.patch


 The DateField component's popup should be styled using Bootstrap 3 mechanisms.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Closed] (TAP5-1007) When Tapestry is loading templates or other files on case-insensitive OSs (Windows) it should trigger an error if the file name case is incorrect (which will result in a ru

2013-09-03 Thread Howard M. Lewis Ship (JIRA)

 [ 
https://issues.apache.org/jira/browse/TAP5-1007?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Howard M. Lewis Ship closed TAP5-1007.
--

Resolution: Fixed

I believe this is now working correctly even on Windows.

 When Tapestry is loading templates or other files on case-insensitive OSs 
 (Windows) it should trigger an error if the file name case is incorrect 
 (which will result in a runtime failure on case-sensitive OSs, such as Linux)
 ---

 Key: TAP5-1007
 URL: https://issues.apache.org/jira/browse/TAP5-1007
 Project: Tapestry 5
  Issue Type: Improvement
  Components: tapestry-core
Affects Versions: 5.2
Reporter: Howard M. Lewis Ship
Assignee: Howard M. Lewis Ship
 Fix For: 5.4


 In other words, on Windows you might find file myComponent.tml when it 
 should be named MyComponent.tml (to match the name of the class, 
 MyComponent).  This is irritating to find in testing or production, and 
 Tapestry should be able to add a check that the case of the file name does 
 not match the expected case.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[1/3] git commit: Add missing btn-default to the cancel button

2013-09-03 Thread hlship
Updated Branches:
  refs/heads/master 2cef4982e - 7171439c1


Add missing btn-default to the cancel button


Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/89ade372
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/89ade372
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/89ade372

Branch: refs/heads/master
Commit: 89ade3722dbe6c5695110945f97635d8c6cd2bb4
Parents: 2cef498
Author: Howard M. Lewis Ship hls...@apache.org
Authored: Tue Sep 3 17:30:38 2013 -0700
Committer: Howard M. Lewis Ship hls...@apache.org
Committed: Tue Sep 3 17:30:38 2013 -0700

--
 .../org/apache/tapestry5/corelib/components/BeanEditForm.tml   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/89ade372/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/BeanEditForm.tml
--
diff --git 
a/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/BeanEditForm.tml
 
b/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/BeanEditForm.tml
index dd17d12..99b747d 100644
--- 
a/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/BeanEditForm.tml
+++ 
b/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/BeanEditForm.tml
@@ -10,7 +10,7 @@
 /div
 t:if test=cancel
 div class=btn-group
-t:submit t:id=cancel mode=cancel class=btn 
value=message:core-cancel-label/
+t:submit t:id=cancel mode=cancel class=btn btn-default 
value=message:core-cancel-label/
 /div
 /t:if
 /div



[2/3] git commit: Make use of FormGroup mixin Modify PropertyEditor to begin/end a Heartbeat, needed by FormGroup mixin

2013-09-03 Thread hlship
Make use of FormGroup mixin
Modify PropertyEditor to begin/end a Heartbeat, needed by FormGroup mixin


Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/c7d47248
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/c7d47248
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/c7d47248

Branch: refs/heads/master
Commit: c7d4724836c5afeea47c22073b371b9f0515b148
Parents: 89ade37
Author: Howard M. Lewis Ship hls...@apache.org
Authored: Tue Sep 3 17:31:12 2013 -0700
Committer: Howard M. Lewis Ship hls...@apache.org
Committed: Tue Sep 3 17:31:12 2013 -0700

--
 .../corelib/components/PropertyEditor.java  |  9 -
 .../corelib/pages/PropertyEditBlocks.tml| 42 
 2 files changed, 16 insertions(+), 35 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/c7d47248/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/PropertyEditor.java
--
diff --git 
a/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/PropertyEditor.java
 
b/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/PropertyEditor.java
index 7a04dc6..c79c192 100644
--- 
a/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/PropertyEditor.java
+++ 
b/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/PropertyEditor.java
@@ -1,4 +1,4 @@
-// Copyright 2007, 2008, 2009, 2011, 2012 The Apache Software Foundation
+// Copyright 2007-2013 The Apache Software Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the License);
 // you may not use this file except in compliance with the License.
@@ -140,6 +140,9 @@ public class PropertyEditor
 @Environmental
 private FormSupport formSupport;
 
+@Inject
+private Heartbeat heartbeat;
+
 private PropertyModel propertyModel;
 
 BeanBlockSource defaultBeanBlockSource()
@@ -217,6 +220,8 @@ public class PropertyEditor
 {
beanValidationContext.setCurrentProperty(propertyName);
 }
+
+heartbeat.begin();
 }
 
 /**
@@ -225,6 +230,8 @@ public class PropertyEditor
  */
 void cleanupEnvironment()
 {
+heartbeat.end();
+
 environment.pop(PropertyEditContext.class);
 }
 

http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/c7d47248/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/pages/PropertyEditBlocks.tml
--
diff --git 
a/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/pages/PropertyEditBlocks.tml
 
b/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/pages/PropertyEditBlocks.tml
index d8e4e12..b22f161 100644
--- 
a/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/pages/PropertyEditBlocks.tml
+++ 
b/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/pages/PropertyEditBlocks.tml
@@ -1,25 +1,16 @@
-div xml:space=default 
xmlns:t=http://tapestry.apache.org/schema/tapestry_5_3.xsd;
+div xmlns:t=http://tapestry.apache.org/schema/tapestry_5_3.xsd;
 
 t:block id=text
-div class=form-group
-t:label for=textField/
-input t:id=textField/
-/div
+input t:mixins=formgroup t:id=textField/
 /t:block
 
 t:block id=number
-div class=form-group
-t:label for=numberField/
-input t:id=numberField class=t-number/
-/div
+input t:id=numberField t:mixins=formgroup class=t-number/
 /t:block
 
 
 t:block id=enum
-div class=form-group
-t:label for=select/
-input t:id=select/
-/div
+input t:id=select t:mixins=formgroup/
 /t:block
 
 t:block id=boolean
@@ -30,40 +21,23 @@
 ${context.label}
 /label
 /div
-
 /t:block
 
 t:block id=date
-
-div class=form-group
-t:label for=dateField/
-input t:id=dateField/
-/div
-
+input t:id=dateField t:mixins=formgroup/
 /t:block
 
 t:block id=calendar
-
-div class=form-group
-t:label for=calendarField/
-input t:id=calendarField/
-/div
-
+input t:id=calendarField t:mixins=formgroup/
 /t:block
 
 t:block id=password
 
-div class=form-group
-t:label for=passwordField/
-input t:id=passwordField/
-/div
+input t:id=passwordField t:mixins=formgroup/
 
 /t:block
 
 t:block id=longtext
-div class=form-group
-t:label for=textarea/
-textarea t:id=textarea/
-/div
+textarea t:id=textarea t:mixins=formgroup/
 /t:block
 /div
\ No newline at 

[3/3] git commit: Ensure that the label rendered by the FormGroup mixin includes the control-label CSS selector

2013-09-03 Thread hlship
Ensure that the label rendered by the FormGroup mixin includes the 
control-label CSS selector


Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/7171439c
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/7171439c
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/7171439c

Branch: refs/heads/master
Commit: 7171439c1e5cc28e11f576c8ca54e78bf54d65eb
Parents: c7d4724
Author: Howard M. Lewis Ship hls...@apache.org
Authored: Tue Sep 3 17:53:14 2013 -0700
Committer: Howard M. Lewis Ship hls...@apache.org
Committed: Tue Sep 3 17:53:14 2013 -0700

--
 .../apache/tapestry5/corelib/mixins/FormGroup.java| 14 --
 .../apache/tapestry5/integration/app1/GridTests.java  |  2 ++
 2 files changed, 14 insertions(+), 2 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/7171439c/tapestry-core/src/main/java/org/apache/tapestry5/corelib/mixins/FormGroup.java
--
diff --git 
a/tapestry-core/src/main/java/org/apache/tapestry5/corelib/mixins/FormGroup.java
 
b/tapestry-core/src/main/java/org/apache/tapestry5/corelib/mixins/FormGroup.java
index 51d2616..2f86675 100644
--- 
a/tapestry-core/src/main/java/org/apache/tapestry5/corelib/mixins/FormGroup.java
+++ 
b/tapestry-core/src/main/java/org/apache/tapestry5/corelib/mixins/FormGroup.java
@@ -16,6 +16,8 @@ package org.apache.tapestry5.corelib.mixins;
 
 import org.apache.tapestry5.Field;
 import org.apache.tapestry5.MarkupWriter;
+import org.apache.tapestry5.ValidationDecorator;
+import org.apache.tapestry5.annotations.Environmental;
 import org.apache.tapestry5.annotations.HeartbeatDeferred;
 import org.apache.tapestry5.annotations.InjectContainer;
 import org.apache.tapestry5.dom.Element;
@@ -37,18 +39,26 @@ import org.apache.tapestry5.dom.Element;
  */
 public class FormGroup
 {
-
 @InjectContainer
 private Field field;
 
 private Element label;
 
+@Environmental
+private ValidationDecorator decorator;
+
 void beginRender(MarkupWriter writer)
 {
 writer.element(div, class, form-group);
-label = writer.element(label);
+
+decorator.beforeLabel(field);
+
+label = writer.element(label, class, control-label);
 writer.end();
+
 fillInLabelAttributes();
+
+decorator.afterLabel(field);
 }
 
 @HeartbeatDeferred

http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/7171439c/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/GridTests.java
--
diff --git 
a/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/GridTests.java
 
b/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/GridTests.java
index a4084b0..69bdbd1 100644
--- 
a/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/GridTests.java
+++ 
b/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/GridTests.java
@@ -16,11 +16,13 @@ package org.apache.tapestry5.integration.app1;
 
 import org.apache.tapestry5.corelib.components.Grid;
 import org.apache.tapestry5.integration.TapestryCoreTestCase;
+import org.apache.tapestry5.test.TapestryTestConfiguration;
 import org.testng.annotations.Test;
 
 /**
  * Tests for the {@link Grid} component.
  */
+@TapestryTestConfiguration(webAppFolder = src/test/app1)
 public class GridTests extends TapestryCoreTestCase
 {
 



Git Push Summary

2013-09-03 Thread hlship
Updated Tags:  refs/tags/5.4-alpha-15 [created] cd7f9c874
  refs/tags/5.4-alpha-16 [created] 7171439c1