You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@royale.apache.org by al...@apache.org on 2018/09/06 05:15:19 UTC

[royale-asjs] branch feature/MXRoyale updated: Update ApplicationDomain.as

This is an automated email from the ASF dual-hosted git repository.

alinakazi pushed a commit to branch feature/MXRoyale
in repository https://gitbox.apache.org/repos/asf/royale-asjs.git


The following commit(s) were added to refs/heads/feature/MXRoyale by this push:
     new aa1b2d8  Update ApplicationDomain.as
aa1b2d8 is described below

commit aa1b2d855088e68d2dddc6e400dfb7281705ab2a
Author: alinakazi <AL...@GMAIL.COM>
AuthorDate: Thu Sep 6 10:15:16 2018 +0500

    Update ApplicationDomain.as
---
 .../src/main/royale/mx/system/ApplicationDomain.as    | 19 ++++++++-----------
 1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDomain.as b/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDomain.as
index 156ac2d..59474bf 100644
--- a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDomain.as
+++ b/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDomain.as
@@ -47,25 +47,22 @@ public class ApplicationDomain extends Object
      *  @playerversion AIR 1.1
      *  @productversion Royale 0.0
      */
-	private var _currentDomain:Object;
-
-	public function get currentDomain():Object
+	
+	
+    public function ApplicationDomain()
     {
-        return _currentDomain;
+        super();
     }
-    
-    public function set currentDomain(obj:Object):void
-    {
-		_currentDomain = obj;
+
+	public static function get currentDomain():ApplicationDomain
+	{
+		return null;
 	}
 	
-	
-	
     public function ApplicationDomain()
     {
         super();
     }
-
 }
 
 }


Re: [royale-asjs] branch feature/MXRoyale updated: Update ApplicationDomain.as

Posted by Alex Harui <ah...@adobe.com.INVALID>.
ApplicationDomain.currentDomain is read-only, so there shouldn’t be a public setter.  Sometimes we hide the setter with protected/private/mx_internal, or we create a  setCurrentDomain function.

It would be great if every committer could spend even an hour helping to get the code to run.  The lowest hanging fruit may be rewriting some of the style property getter/setters to use ValueManager.  You can see some of that has already been done in UIComponent.

I think we still need someone to write a ColorPicker that looks sort of like the Flex one.  It would be written for the Basic component set with the standard beads and then the beads would be repurposed in the MXRoyale set.

Another "simple" task should be "copying" the MXRoyale UI widget implementations into SparkRoyale.  I put "copying" in quotes because it isn't a straight copy.  You have to have a basic understanding of the differences between MX and Spark (other than skinning).  For example, you wouldn't copy the text widgets because the Spark ones use TLF and I think I have those done mostly already.  But Spark List can probably copy lots of things from MX List.  And DataGrid, and so on.  For the purposes of migration, I don't think we really need to have two truly different component architectures for MX and Spark.  They can both re-use the same Basic beads, IMO.

Thanks,
-Alex



On 9/6/18, 9:31 AM, "Harbs" <ha...@gmail.com> wrote:

    I get that. I was just curious why the setter was removed…
    
    Either way, if there’s some low hanging fruit in the emulation that you think I should work on, let me know… I don’t have a lot of time right now, but I’m happy to help with what I can.
    
    Harbs
    
    > On Sep 6, 2018, at 7:19 PM, Alex Harui <ah...@adobe.com.INVALID> wrote:
    > 
    > Alina's primary goal is to get all of her code to compile, so many emulation APIs are currently stubs that return null or 0 or "".  
    > 
    > My goal is to get this stuff to run, so I fill in the method bodies with real code.  Hopefully other committers will pitch in soon.
    > 
    > -Alex
    > 
    > On 9/6/18, 9:17 AM, "Harbs" <harbs.lists@gmail.com <ma...@gmail.com>> wrote:
    > 
    >    Dunno. It just seems odd to me that it was changed to always return null.
    > 
    >    I was just asking. I was not following the work on this closely enough.
    > 
    >> On Sep 6, 2018, at 6:55 PM, Alex Harui <aharui@adobe.com.INVALID <ma...@adobe.com.INVALID>> wrote:
    >> 
    >> Harbs, what was wrong with this commit?  It looked ok to me.  currentDomain is static.
    >> 
    >> -Alex
    >> 
    >> On 9/6/18, 2:27 AM, "Alina Kazi" <alina.kazi@d-bz.com <ma...@d-bz.com> <mailto:alina.kazi@d-bz.com <ma...@d-bz.com>>> wrote:
    >> 
    >>   Reverting this Commit. 
    >> 
    >>   -----Original Message-----
    >>   From: Harbs [mailto:harbs.lists@gmail.com <ma...@gmail.com>] 
    >>   Sent: Thursday, September 06, 2018 1:08 PM
    >>   To: dev@royale.apache.org <ma...@royale.apache.org>
    >>   Subject: Re: [royale-asjs] branch feature/MXRoyale updated: Update
    >>   ApplicationDomain.as
    >> 
    >>   Why was the currentDomain setter removed?
    >> 
    >>> On Sep 6, 2018, at 8:15 AM, alinakazi@apache.org <ma...@apache.org> wrote:
    >>> 
    >>> This is an automated email from the ASF dual-hosted git repository.
    >>> 
    >>> alinakazi pushed a commit to branch feature/MXRoyale in repository 
    >>> https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C04e7400077f0460e460108d614163040%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718482856290457&amp;sdata=gBWMhaCkX0fls5gbzNWpeZp2t%2BPLvveUNKDMCB8zWOc%3D&amp;reserved=0 <https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C04e7400077f0460e460108d614163040%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718482856290457&amp;sdata=gBWMhaCkX0fls5gbzNWpeZp2t%2BPLvveUNKDMCB8zWOc%3D&amp;reserved=0> <https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C04e7400077f0460e460108d614163040%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718482856290457&amp;sdata=gBWMhaCkX0fls5gbzNWpeZp2t%2BPLvveUNKDMCB8zWOc%3D&amp;reserved=0 <https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C04e7400077f0460e460108d614163040%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718482856290457&amp;sdata=gBWMhaCkX0fls5gbzNWpeZp2t%2BPLvveUNKDMCB8zWOc%3D&amp;reserved=0>>
    >>> 
    >>> 
    >>> The following commit(s) were added to refs/heads/feature/MXRoyale by this
    >>   push:
    >>>   new aa1b2d8  Update ApplicationDomain.as
    >>> aa1b2d8 is described below
    >>> 
    >>> commit aa1b2d855088e68d2dddc6e400dfb7281705ab2a
    >>> Author: alinakazi <ALINAKAZI1990@GMAIL.COM <ma...@GMAIL.COM>>
    >>> AuthorDate: Thu Sep 6 10:15:16 2018 +0500
    >>> 
    >>>  Update ApplicationDomain.as
    >>> ---
    >>> .../src/main/royale/mx/system/ApplicationDomain.as    | 19
    >>   ++++++++-----------
    >>> 1 file changed, 8 insertions(+), 11 deletions(-)
    >>> 
    >>> diff --git 
    >>> a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
    >>> main.as 
    >>> b/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
    >>> main.as
    >>> index 156ac2d..59474bf 100644
    >>> --- 
    >>> a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
    >>> main.as
    >>> +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/system/Applicati
    >>> +++ onDomain.as
    >>> @@ -47,25 +47,22 @@ public class ApplicationDomain extends Object
    >>>    *  @playerversion AIR 1.1
    >>>    *  @productversion Royale 0.0
    >>>    */
    >>> -	private var _currentDomain:Object;
    >>> -
    >>> -	public function get currentDomain():Object
    >>> +	
    >>> +	
    >>> +    public function ApplicationDomain()
    >>>   {
    >>> -        return _currentDomain;
    >>> +        super();
    >>>   }
    >>> -    
    >>> -    public function set currentDomain(obj:Object):void
    >>> -    {
    >>> -		_currentDomain = obj;
    >>> +
    >>> +	public static function get currentDomain():ApplicationDomain
    >>> +	{
    >>> +		return null;
    >>> 	}
    >>> 	
    >>> -	
    >>> -	
    >>>   public function ApplicationDomain()
    >>>   {
    >>>       super();
    >>>   }
    >>> -
    >>> }
    >>> 
    >>> }
    
    


Re: [royale-asjs] branch feature/MXRoyale updated: Update ApplicationDomain.as

Posted by Harbs <ha...@gmail.com>.
I get that. I was just curious why the setter was removed…

Either way, if there’s some low hanging fruit in the emulation that you think I should work on, let me know… I don’t have a lot of time right now, but I’m happy to help with what I can.

Harbs

> On Sep 6, 2018, at 7:19 PM, Alex Harui <ah...@adobe.com.INVALID> wrote:
> 
> Alina's primary goal is to get all of her code to compile, so many emulation APIs are currently stubs that return null or 0 or "".  
> 
> My goal is to get this stuff to run, so I fill in the method bodies with real code.  Hopefully other committers will pitch in soon.
> 
> -Alex
> 
> On 9/6/18, 9:17 AM, "Harbs" <harbs.lists@gmail.com <ma...@gmail.com>> wrote:
> 
>    Dunno. It just seems odd to me that it was changed to always return null.
> 
>    I was just asking. I was not following the work on this closely enough.
> 
>> On Sep 6, 2018, at 6:55 PM, Alex Harui <aharui@adobe.com.INVALID <ma...@adobe.com.INVALID>> wrote:
>> 
>> Harbs, what was wrong with this commit?  It looked ok to me.  currentDomain is static.
>> 
>> -Alex
>> 
>> On 9/6/18, 2:27 AM, "Alina Kazi" <alina.kazi@d-bz.com <ma...@d-bz.com> <mailto:alina.kazi@d-bz.com <ma...@d-bz.com>>> wrote:
>> 
>>   Reverting this Commit. 
>> 
>>   -----Original Message-----
>>   From: Harbs [mailto:harbs.lists@gmail.com <ma...@gmail.com>] 
>>   Sent: Thursday, September 06, 2018 1:08 PM
>>   To: dev@royale.apache.org <ma...@royale.apache.org>
>>   Subject: Re: [royale-asjs] branch feature/MXRoyale updated: Update
>>   ApplicationDomain.as
>> 
>>   Why was the currentDomain setter removed?
>> 
>>> On Sep 6, 2018, at 8:15 AM, alinakazi@apache.org <ma...@apache.org> wrote:
>>> 
>>> This is an automated email from the ASF dual-hosted git repository.
>>> 
>>> alinakazi pushed a commit to branch feature/MXRoyale in repository 
>>> https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C133b6ea757c849b9a69308d614143aa5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718474443813011&amp;sdata=gjL3N4gZJkwiW%2Bj9w2Og%2BQnt93CcSmzHTdgBgrgyWhE%3D&amp;reserved=0 <https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C133b6ea757c849b9a69308d614143aa5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718474443813011&amp;sdata=gjL3N4gZJkwiW%2Bj9w2Og%2BQnt93CcSmzHTdgBgrgyWhE%3D&amp;reserved=0> <https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C133b6ea757c849b9a69308d614143aa5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718474443813011&amp;sdata=gjL3N4gZJkwiW%2Bj9w2Og%2BQnt93CcSmzHTdgBgrgyWhE%3D&amp;reserved=0 <https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C133b6ea757c849b9a69308d614143aa5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718474443813011&amp;sdata=gjL3N4gZJkwiW%2Bj9w2Og%2BQnt93CcSmzHTdgBgrgyWhE%3D&amp;reserved=0>>
>>> 
>>> 
>>> The following commit(s) were added to refs/heads/feature/MXRoyale by this
>>   push:
>>>   new aa1b2d8  Update ApplicationDomain.as
>>> aa1b2d8 is described below
>>> 
>>> commit aa1b2d855088e68d2dddc6e400dfb7281705ab2a
>>> Author: alinakazi <ALINAKAZI1990@GMAIL.COM <ma...@GMAIL.COM>>
>>> AuthorDate: Thu Sep 6 10:15:16 2018 +0500
>>> 
>>>  Update ApplicationDomain.as
>>> ---
>>> .../src/main/royale/mx/system/ApplicationDomain.as    | 19
>>   ++++++++-----------
>>> 1 file changed, 8 insertions(+), 11 deletions(-)
>>> 
>>> diff --git 
>>> a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
>>> main.as 
>>> b/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
>>> main.as
>>> index 156ac2d..59474bf 100644
>>> --- 
>>> a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
>>> main.as
>>> +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/system/Applicati
>>> +++ onDomain.as
>>> @@ -47,25 +47,22 @@ public class ApplicationDomain extends Object
>>>    *  @playerversion AIR 1.1
>>>    *  @productversion Royale 0.0
>>>    */
>>> -	private var _currentDomain:Object;
>>> -
>>> -	public function get currentDomain():Object
>>> +	
>>> +	
>>> +    public function ApplicationDomain()
>>>   {
>>> -        return _currentDomain;
>>> +        super();
>>>   }
>>> -    
>>> -    public function set currentDomain(obj:Object):void
>>> -    {
>>> -		_currentDomain = obj;
>>> +
>>> +	public static function get currentDomain():ApplicationDomain
>>> +	{
>>> +		return null;
>>> 	}
>>> 	
>>> -	
>>> -	
>>>   public function ApplicationDomain()
>>>   {
>>>       super();
>>>   }
>>> -
>>> }
>>> 
>>> }


Re: [royale-asjs] branch feature/MXRoyale updated: Update ApplicationDomain.as

Posted by Alex Harui <ah...@adobe.com.INVALID>.
Alina's primary goal is to get all of her code to compile, so many emulation APIs are currently stubs that return null or 0 or "".  

My goal is to get this stuff to run, so I fill in the method bodies with real code.  Hopefully other committers will pitch in soon.

-Alex

On 9/6/18, 9:17 AM, "Harbs" <ha...@gmail.com> wrote:

    Dunno. It just seems odd to me that it was changed to always return null.
    
    I was just asking. I was not following the work on this closely enough.
    
    > On Sep 6, 2018, at 6:55 PM, Alex Harui <ah...@adobe.com.INVALID> wrote:
    > 
    > Harbs, what was wrong with this commit?  It looked ok to me.  currentDomain is static.
    > 
    > -Alex
    > 
    > On 9/6/18, 2:27 AM, "Alina Kazi" <alina.kazi@d-bz.com <ma...@d-bz.com>> wrote:
    > 
    >    Reverting this Commit. 
    > 
    >    -----Original Message-----
    >    From: Harbs [mailto:harbs.lists@gmail.com] 
    >    Sent: Thursday, September 06, 2018 1:08 PM
    >    To: dev@royale.apache.org
    >    Subject: Re: [royale-asjs] branch feature/MXRoyale updated: Update
    >    ApplicationDomain.as
    > 
    >    Why was the currentDomain setter removed?
    > 
    >> On Sep 6, 2018, at 8:15 AM, alinakazi@apache.org wrote:
    >> 
    >> This is an automated email from the ASF dual-hosted git repository.
    >> 
    >> alinakazi pushed a commit to branch feature/MXRoyale in repository 
    >> https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C133b6ea757c849b9a69308d614143aa5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718474443813011&amp;sdata=gjL3N4gZJkwiW%2Bj9w2Og%2BQnt93CcSmzHTdgBgrgyWhE%3D&amp;reserved=0 <https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C133b6ea757c849b9a69308d614143aa5%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718474443813011&amp;sdata=gjL3N4gZJkwiW%2Bj9w2Og%2BQnt93CcSmzHTdgBgrgyWhE%3D&amp;reserved=0>
    >> 
    >> 
    >> The following commit(s) were added to refs/heads/feature/MXRoyale by this
    >    push:
    >>    new aa1b2d8  Update ApplicationDomain.as
    >> aa1b2d8 is described below
    >> 
    >> commit aa1b2d855088e68d2dddc6e400dfb7281705ab2a
    >> Author: alinakazi <AL...@GMAIL.COM>
    >> AuthorDate: Thu Sep 6 10:15:16 2018 +0500
    >> 
    >>   Update ApplicationDomain.as
    >> ---
    >> .../src/main/royale/mx/system/ApplicationDomain.as    | 19
    >    ++++++++-----------
    >> 1 file changed, 8 insertions(+), 11 deletions(-)
    >> 
    >> diff --git 
    >> a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
    >> main.as 
    >> b/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
    >> main.as
    >> index 156ac2d..59474bf 100644
    >> --- 
    >> a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
    >> main.as
    >> +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/system/Applicati
    >> +++ onDomain.as
    >> @@ -47,25 +47,22 @@ public class ApplicationDomain extends Object
    >>     *  @playerversion AIR 1.1
    >>     *  @productversion Royale 0.0
    >>     */
    >> -	private var _currentDomain:Object;
    >> -
    >> -	public function get currentDomain():Object
    >> +	
    >> +	
    >> +    public function ApplicationDomain()
    >>    {
    >> -        return _currentDomain;
    >> +        super();
    >>    }
    >> -    
    >> -    public function set currentDomain(obj:Object):void
    >> -    {
    >> -		_currentDomain = obj;
    >> +
    >> +	public static function get currentDomain():ApplicationDomain
    >> +	{
    >> +		return null;
    >> 	}
    >> 	
    >> -	
    >> -	
    >>    public function ApplicationDomain()
    >>    {
    >>        super();
    >>    }
    >> -
    >> }
    >> 
    >> }
    
    


Re: [royale-asjs] branch feature/MXRoyale updated: Update ApplicationDomain.as

Posted by Harbs <ha...@gmail.com>.
Dunno. It just seems odd to me that it was changed to always return null.

I was just asking. I was not following the work on this closely enough.

> On Sep 6, 2018, at 6:55 PM, Alex Harui <ah...@adobe.com.INVALID> wrote:
> 
> Harbs, what was wrong with this commit?  It looked ok to me.  currentDomain is static.
> 
> -Alex
> 
> On 9/6/18, 2:27 AM, "Alina Kazi" <alina.kazi@d-bz.com <ma...@d-bz.com>> wrote:
> 
>    Reverting this Commit. 
> 
>    -----Original Message-----
>    From: Harbs [mailto:harbs.lists@gmail.com] 
>    Sent: Thursday, September 06, 2018 1:08 PM
>    To: dev@royale.apache.org
>    Subject: Re: [royale-asjs] branch feature/MXRoyale updated: Update
>    ApplicationDomain.as
> 
>    Why was the currentDomain setter removed?
> 
>> On Sep 6, 2018, at 8:15 AM, alinakazi@apache.org wrote:
>> 
>> This is an automated email from the ASF dual-hosted git repository.
>> 
>> alinakazi pushed a commit to branch feature/MXRoyale in repository 
>> https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cdc54f95ff680416e403308d613db0798%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718228774046734&amp;sdata=MPJsY9NB5fAp2dxzuKhowR1I6kkcYIqQKLTt4TGpxzg%3D&amp;reserved=0 <https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cdc54f95ff680416e403308d613db0798%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718228774046734&amp;sdata=MPJsY9NB5fAp2dxzuKhowR1I6kkcYIqQKLTt4TGpxzg%3D&amp;reserved=0>
>> 
>> 
>> The following commit(s) were added to refs/heads/feature/MXRoyale by this
>    push:
>>    new aa1b2d8  Update ApplicationDomain.as
>> aa1b2d8 is described below
>> 
>> commit aa1b2d855088e68d2dddc6e400dfb7281705ab2a
>> Author: alinakazi <AL...@GMAIL.COM>
>> AuthorDate: Thu Sep 6 10:15:16 2018 +0500
>> 
>>   Update ApplicationDomain.as
>> ---
>> .../src/main/royale/mx/system/ApplicationDomain.as    | 19
>    ++++++++-----------
>> 1 file changed, 8 insertions(+), 11 deletions(-)
>> 
>> diff --git 
>> a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
>> main.as 
>> b/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
>> main.as
>> index 156ac2d..59474bf 100644
>> --- 
>> a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
>> main.as
>> +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/system/Applicati
>> +++ onDomain.as
>> @@ -47,25 +47,22 @@ public class ApplicationDomain extends Object
>>     *  @playerversion AIR 1.1
>>     *  @productversion Royale 0.0
>>     */
>> -	private var _currentDomain:Object;
>> -
>> -	public function get currentDomain():Object
>> +	
>> +	
>> +    public function ApplicationDomain()
>>    {
>> -        return _currentDomain;
>> +        super();
>>    }
>> -    
>> -    public function set currentDomain(obj:Object):void
>> -    {
>> -		_currentDomain = obj;
>> +
>> +	public static function get currentDomain():ApplicationDomain
>> +	{
>> +		return null;
>> 	}
>> 	
>> -	
>> -	
>>    public function ApplicationDomain()
>>    {
>>        super();
>>    }
>> -
>> }
>> 
>> }


Re: [royale-asjs] branch feature/MXRoyale updated: Update ApplicationDomain.as

Posted by Alex Harui <ah...@adobe.com.INVALID>.
Harbs, what was wrong with this commit?  It looked ok to me.  currentDomain is static.

-Alex

On 9/6/18, 2:27 AM, "Alina Kazi" <al...@d-bz.com> wrote:

    Reverting this Commit. 
    
    -----Original Message-----
    From: Harbs [mailto:harbs.lists@gmail.com] 
    Sent: Thursday, September 06, 2018 1:08 PM
    To: dev@royale.apache.org
    Subject: Re: [royale-asjs] branch feature/MXRoyale updated: Update
    ApplicationDomain.as
    
    Why was the currentDomain setter removed?
    
    > On Sep 6, 2018, at 8:15 AM, alinakazi@apache.org wrote:
    > 
    > This is an automated email from the ASF dual-hosted git repository.
    > 
    > alinakazi pushed a commit to branch feature/MXRoyale in repository 
    > https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cdc54f95ff680416e403308d613db0798%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636718228774046734&amp;sdata=MPJsY9NB5fAp2dxzuKhowR1I6kkcYIqQKLTt4TGpxzg%3D&amp;reserved=0
    > 
    > 
    > The following commit(s) were added to refs/heads/feature/MXRoyale by this
    push:
    >     new aa1b2d8  Update ApplicationDomain.as
    > aa1b2d8 is described below
    > 
    > commit aa1b2d855088e68d2dddc6e400dfb7281705ab2a
    > Author: alinakazi <AL...@GMAIL.COM>
    > AuthorDate: Thu Sep 6 10:15:16 2018 +0500
    > 
    >    Update ApplicationDomain.as
    > ---
    > .../src/main/royale/mx/system/ApplicationDomain.as    | 19
    ++++++++-----------
    > 1 file changed, 8 insertions(+), 11 deletions(-)
    > 
    > diff --git 
    > a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
    > main.as 
    > b/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
    > main.as
    > index 156ac2d..59474bf 100644
    > --- 
    > a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
    > main.as
    > +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/system/Applicati
    > +++ onDomain.as
    > @@ -47,25 +47,22 @@ public class ApplicationDomain extends Object
    >      *  @playerversion AIR 1.1
    >      *  @productversion Royale 0.0
    >      */
    > -	private var _currentDomain:Object;
    > -
    > -	public function get currentDomain():Object
    > +	
    > +	
    > +    public function ApplicationDomain()
    >     {
    > -        return _currentDomain;
    > +        super();
    >     }
    > -    
    > -    public function set currentDomain(obj:Object):void
    > -    {
    > -		_currentDomain = obj;
    > +
    > +	public static function get currentDomain():ApplicationDomain
    > +	{
    > +		return null;
    > 	}
    > 	
    > -	
    > -	
    >     public function ApplicationDomain()
    >     {
    >         super();
    >     }
    > -
    > }
    > 
    > }
    > 
    
    
    


RE: [royale-asjs] branch feature/MXRoyale updated: Update ApplicationDomain.as

Posted by Alina Kazi <al...@d-bz.com>.
Reverting this Commit. 

-----Original Message-----
From: Harbs [mailto:harbs.lists@gmail.com] 
Sent: Thursday, September 06, 2018 1:08 PM
To: dev@royale.apache.org
Subject: Re: [royale-asjs] branch feature/MXRoyale updated: Update
ApplicationDomain.as

Why was the currentDomain setter removed?

> On Sep 6, 2018, at 8:15 AM, alinakazi@apache.org wrote:
> 
> This is an automated email from the ASF dual-hosted git repository.
> 
> alinakazi pushed a commit to branch feature/MXRoyale in repository 
> https://gitbox.apache.org/repos/asf/royale-asjs.git
> 
> 
> The following commit(s) were added to refs/heads/feature/MXRoyale by this
push:
>     new aa1b2d8  Update ApplicationDomain.as
> aa1b2d8 is described below
> 
> commit aa1b2d855088e68d2dddc6e400dfb7281705ab2a
> Author: alinakazi <AL...@GMAIL.COM>
> AuthorDate: Thu Sep 6 10:15:16 2018 +0500
> 
>    Update ApplicationDomain.as
> ---
> .../src/main/royale/mx/system/ApplicationDomain.as    | 19
++++++++-----------
> 1 file changed, 8 insertions(+), 11 deletions(-)
> 
> diff --git 
> a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
> main.as 
> b/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
> main.as
> index 156ac2d..59474bf 100644
> --- 
> a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDo
> main.as
> +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/system/Applicati
> +++ onDomain.as
> @@ -47,25 +47,22 @@ public class ApplicationDomain extends Object
>      *  @playerversion AIR 1.1
>      *  @productversion Royale 0.0
>      */
> -	private var _currentDomain:Object;
> -
> -	public function get currentDomain():Object
> +	
> +	
> +    public function ApplicationDomain()
>     {
> -        return _currentDomain;
> +        super();
>     }
> -    
> -    public function set currentDomain(obj:Object):void
> -    {
> -		_currentDomain = obj;
> +
> +	public static function get currentDomain():ApplicationDomain
> +	{
> +		return null;
> 	}
> 	
> -	
> -	
>     public function ApplicationDomain()
>     {
>         super();
>     }
> -
> }
> 
> }
> 



Re: [royale-asjs] branch feature/MXRoyale updated: Update ApplicationDomain.as

Posted by Harbs <ha...@gmail.com>.
Why was the currentDomain setter removed?

> On Sep 6, 2018, at 8:15 AM, alinakazi@apache.org wrote:
> 
> This is an automated email from the ASF dual-hosted git repository.
> 
> alinakazi pushed a commit to branch feature/MXRoyale
> in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
> 
> 
> The following commit(s) were added to refs/heads/feature/MXRoyale by this push:
>     new aa1b2d8  Update ApplicationDomain.as
> aa1b2d8 is described below
> 
> commit aa1b2d855088e68d2dddc6e400dfb7281705ab2a
> Author: alinakazi <AL...@GMAIL.COM>
> AuthorDate: Thu Sep 6 10:15:16 2018 +0500
> 
>    Update ApplicationDomain.as
> ---
> .../src/main/royale/mx/system/ApplicationDomain.as    | 19 ++++++++-----------
> 1 file changed, 8 insertions(+), 11 deletions(-)
> 
> diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDomain.as b/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDomain.as
> index 156ac2d..59474bf 100644
> --- a/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDomain.as
> +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/system/ApplicationDomain.as
> @@ -47,25 +47,22 @@ public class ApplicationDomain extends Object
>      *  @playerversion AIR 1.1
>      *  @productversion Royale 0.0
>      */
> -	private var _currentDomain:Object;
> -
> -	public function get currentDomain():Object
> +	
> +	
> +    public function ApplicationDomain()
>     {
> -        return _currentDomain;
> +        super();
>     }
> -    
> -    public function set currentDomain(obj:Object):void
> -    {
> -		_currentDomain = obj;
> +
> +	public static function get currentDomain():ApplicationDomain
> +	{
> +		return null;
> 	}
> 	
> -	
> -	
>     public function ApplicationDomain()
>     {
>         super();
>     }
> -
> }
> 
> }
>