You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by ka...@apache.org on 2013/04/15 15:19:32 UTC
git commit: FIXED - TAP5-2025: Duplicate generated ids - use nanoTime
instead of millis to avoid id conflicts
Updated Branches:
refs/heads/master ac4dabeea -> 6fd011e7d
FIXED - TAP5-2025: Duplicate generated ids
- use nanoTime instead of millis to avoid id conflicts
Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/6fd011e7
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/6fd011e7
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/6fd011e7
Branch: refs/heads/master
Commit: 6fd011e7d598a087766e27bc89bdc239dc60dc6d
Parents: ac4dabe
Author: kaosko <ka...@apache.org>
Authored: Mon Apr 15 05:57:28 2013 -0700
Committer: kaosko <ka...@apache.org>
Committed: Mon Apr 15 14:18:13 2013 +0100
----------------------------------------------------------------------
.../ajax/AjaxFormUpdateControllerImpl.java | 2 +-
.../services/javascript/JavaScriptModule.java | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/6fd011e7/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/ajax/AjaxFormUpdateControllerImpl.java
----------------------------------------------------------------------
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/ajax/AjaxFormUpdateControllerImpl.java b/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/ajax/AjaxFormUpdateControllerImpl.java
index 6bcfadc..ae75c7c 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/ajax/AjaxFormUpdateControllerImpl.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/ajax/AjaxFormUpdateControllerImpl.java
@@ -134,7 +134,7 @@ public class AjaxFormUpdateControllerImpl implements AjaxFormUpdateController
// JavaScriptSupport's job). It would be nice if we could agree on the uid, but
// not essential.
- String uid = Long.toHexString(System.currentTimeMillis());
+ String uid = Long.toHexString(System.nanoTime());
IdAllocator idAllocator = new IdAllocator("_" + uid);
http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/6fd011e7/tapestry-core/src/main/java/org/apache/tapestry5/services/javascript/JavaScriptModule.java
----------------------------------------------------------------------
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/services/javascript/JavaScriptModule.java b/tapestry-core/src/main/java/org/apache/tapestry5/services/javascript/JavaScriptModule.java
index b6a08ee..cd19a5e 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/services/javascript/JavaScriptModule.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/services/javascript/JavaScriptModule.java
@@ -186,7 +186,7 @@ public class JavaScriptModule
{
public void renderMarkup(MarkupWriter writer, JSONObject reply, PartialMarkupRenderer renderer)
{
- String uid = Long.toHexString(System.currentTimeMillis());
+ String uid = Long.toHexString(System.nanoTime());
String namespace = "_" + uid;