You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flex.apache.org by ah...@apache.org on 2013/05/05 08:16:51 UTC

[1/2] git commit: [flex-asjs] [refs/heads/develop] - Fix setting selectedIndex and selectedItem in List classes

Updated Branches:
  refs/heads/develop a34f288ab -> 01b9aae25


Fix setting selectedIndex and selectedItem in List classes


Project: http://git-wip-us.apache.org/repos/asf/flex-asjs/repo
Commit: http://git-wip-us.apache.org/repos/asf/flex-asjs/commit/ca117131
Tree: http://git-wip-us.apache.org/repos/asf/flex-asjs/tree/ca117131
Diff: http://git-wip-us.apache.org/repos/asf/flex-asjs/diff/ca117131

Branch: refs/heads/develop
Commit: ca117131db5b0eef45d2271d32aa91f7938cd3ca
Parents: a34f288
Author: Alex Harui <ah...@apache.org>
Authored: Sat May 4 23:15:32 2013 -0700
Committer: Alex Harui <ah...@apache.org>
Committed: Sat May 4 23:15:32 2013 -0700

----------------------------------------------------------------------
 .../js/FlexJS/src/org/apache/flex/core/ListBase.js |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flex-asjs/blob/ca117131/frameworks/js/FlexJS/src/org/apache/flex/core/ListBase.js
----------------------------------------------------------------------
diff --git a/frameworks/js/FlexJS/src/org/apache/flex/core/ListBase.js b/frameworks/js/FlexJS/src/org/apache/flex/core/ListBase.js
index 5a2fd27..275a679 100644
--- a/frameworks/js/FlexJS/src/org/apache/flex/core/ListBase.js
+++ b/frameworks/js/FlexJS/src/org/apache/flex/core/ListBase.js
@@ -122,7 +122,7 @@ org.apache.flex.core.ListBase.prototype.set_selectedIndex =
     function(value) {
   this.selectedIndex_ = value;
 
-  if (this.element.selectedIndex) {
+  if (this.element.selectedIndex !== undefined) {
     this.element.selectedIndex = value;
   }
 };
@@ -168,7 +168,7 @@ org.apache.flex.core.ListBase.prototype.set_selectedItem =
   if (i < n) {
     this.selectedIndex_ = i;
 
-    if (this.element.selectedIndex) {
+    if (this.element.selectedIndex !== undefined) {
       this.element.selectedIndex = i;
     }
 


[2/2] git commit: [flex-asjs] [refs/heads/develop] - add get_target() because codegen doesn't know that org.apache.flex.events.Event is wrapping goog.events.Event which has target as a property instead of a getter

Posted by ah...@apache.org.
add get_target() because codegen doesn't know that org.apache.flex.events.Event is wrapping goog.events.Event which has target as a property instead of a getter


Project: http://git-wip-us.apache.org/repos/asf/flex-asjs/repo
Commit: http://git-wip-us.apache.org/repos/asf/flex-asjs/commit/01b9aae2
Tree: http://git-wip-us.apache.org/repos/asf/flex-asjs/tree/01b9aae2
Diff: http://git-wip-us.apache.org/repos/asf/flex-asjs/diff/01b9aae2

Branch: refs/heads/develop
Commit: 01b9aae25d38c4749514d92b92f0b49de637594c
Parents: ca11713
Author: Alex Harui <ah...@apache.org>
Authored: Sat May 4 23:16:35 2013 -0700
Committer: Alex Harui <ah...@apache.org>
Committed: Sat May 4 23:16:35 2013 -0700

----------------------------------------------------------------------
 .../js/FlexJS/src/org/apache/flex/events/Event.js  |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flex-asjs/blob/01b9aae2/frameworks/js/FlexJS/src/org/apache/flex/events/Event.js
----------------------------------------------------------------------
diff --git a/frameworks/js/FlexJS/src/org/apache/flex/events/Event.js b/frameworks/js/FlexJS/src/org/apache/flex/events/Event.js
index 51752cc..2c7900f 100644
--- a/frameworks/js/FlexJS/src/org/apache/flex/events/Event.js
+++ b/frameworks/js/FlexJS/src/org/apache/flex/events/Event.js
@@ -41,6 +41,14 @@ org.apache.flex.events.Event.prototype.init = function(type) {
   this.type = type;
 };
 
+/**
+ * @expose
+ * @this {org.apache.flex.events.Event}
+ * @return {Object} The event target.
+ */
+org.apache.flex.events.Event.prototype.get_target = function() {
+  return this.target;
+};
 
 /**
  * @expose