I will try to fix the compiler. The only thing I saw in the commit that didn't look right was assigning the itemrenderer's data to the result of the labelFunction. I don't think Flex ever replaced the data property, I think the text to display was computed elsewhere.
-Alex On 10/28/19, 12:20 AM, "Piotr Zarzycki" <[email protected]> wrote: Alex, Will you look into compiler issue ? pon., 28 paź 2019 o 07:58 Piotr Zarzycki <[email protected]> napisał(a): > What do you think about implementation ? Cause I would like to merge it > this week, even if there is compiler issue. > > pon., 28 paź 2019 o 07:12 Alex Harui <[email protected]> > napisał(a): > >> Yes, looks like the compiler is not handling attributes of type Function >> in MXML correctly. >> >> On 10/27/19, 7:56 AM, "Piotr Zarzycki" <[email protected]> >> wrote: >> >> Hi Alex, >> >> I have pushed implementation and described it here [1]. Take a look >> and let >> me know if it's ok. It doesn't work - I'm thinking that it may be >> some kind >> of compiler issue. >> >> [1] >> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-asjs%2Fissues%2F472%23issuecomment-546701359&data=02%7C01%7Caharui%40adobe.com%7Cdd2d1fd4495844536a9d08d75b7741ae%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637078440071742110&sdata=mzHO5rR5nJ%2BzGpnfO54zJDY76ZcIfArPA4ctY1z%2BRsA%3D&reserved=0 >> >> Thanks, >> Piotr >> >> wt., 22 paź 2019 o 19:31 Alex Harui <[email protected]> >> napisał(a): >> >> > The emulation is currently only promising to support public APIs. >> If it >> > turns out that lots of folks have subclasses that call >> makeRowsAndColumns >> > then we'll emulate it. >> > >> > I recommend trying to extend the labelField pattern in Basic. IMO, >> the >> > labelField pattern is: >> > -"labelField" property on top-level component (TLC) passes to model >> > -Factory passes labelField property from model to itemRenderer >> > -ItemRenderer decides what to do with labelFunction >> > >> > So, in keeping with PAYG, whenever we get around to supporting >> > labelFunction in Basic, we'd add a code to subclasses of the TLCs, >> the >> > models, the factories, and the item renderers. >> > >> > I think that's a different flow from Flex, but should be sufficient >> unless >> > folks are relying on labelFunction/labelField in the .text property >> of the >> > listData structures. If you run into that then we can discuss how >> to >> > emulate the listData structure. >> > >> > My 2 cents, >> > -Alex >> > >> > On 10/22/19, 4:15 AM, "Yishay Weiss" <[email protected]> >> wrote: >> > >> > In my opinion it’s easier and more elegant to add the missing >> > functionality in a bead (e.g. >> > DataItemRendererFactoryForICollectionViewAdvancedDataGridData ) >> rather than >> > reproduce Flex’s class hierarchy. The only reason to do the latter >> would be >> > to make code that depends on a particular class hierarchy (is >> operator, >> > casting) run correctly. >> > >> > My approach would be to add the functionality in a bead, but >> make that >> > bead reusable so that it can be referenced from AdvancedListBase in >> case >> > that’s necessary. >> > >> > I hope that makes sense. >> > >> > ________________________________ >> > From: Piotr Zarzycki <[email protected]> >> > Sent: Tuesday, October 22, 2019 12:48:52 PM >> > To: [email protected] <[email protected]> >> > Subject: Emulation of labelFunction in ADG >> > >> > Hi Guys, >> > >> > I'm working on fixing issue [1]. labelFunction wasn't emulated >> at all >> > so >> > I'm trying to figure out how to add that. >> > >> > I checked deeper Flex code and it looks like launching >> labelFunction is >> > starting to happen during row creation. There is >> > function makeRowsAndColumnsWithExtraRows in AdvancedListBase >> where >> > everything is starting. >> > >> > My question is - should I go and start copying code and classes >> to fill >> > that gap or rather place launching that formatting in >> > DataItemRendererFactoryForICollectionViewAdvancedDataGridData >> > where we are creating and setup renderer ? What is the general >> > approach. >> > >> > [1] >> > >> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-asjs%2Fissues%2F472&data=02%7C01%7Caharui%40adobe.com%7Cdd2d1fd4495844536a9d08d75b7741ae%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637078440071742110&sdata=u012BCFT7iVQmuXnePFEeV0HSd3kG2rU9LO7I8RBhrI%3D&reserved=0 >> > >> > Thanks, >> > -- >> > >> > Piotr Zarzycki >> > >> > Patreon: * >> > >> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7Cdd2d1fd4495844536a9d08d75b7741ae%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637078440071742110&sdata=zOA%2FU%2BXBiwodptKThzDDrkWHE1%2FblZzz5EcJs%2BV23o4%3D&reserved=0 >> > < >> > >> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7Cdd2d1fd4495844536a9d08d75b7741ae%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637078440071742110&sdata=zOA%2FU%2BXBiwodptKThzDDrkWHE1%2FblZzz5EcJs%2BV23o4%3D&reserved=0 >> > >* >> > >> > >> > >> >> -- >> >> Piotr Zarzycki >> >> Patreon: * >> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7Cdd2d1fd4495844536a9d08d75b7741ae%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637078440071742110&sdata=zOA%2FU%2BXBiwodptKThzDDrkWHE1%2FblZzz5EcJs%2BV23o4%3D&reserved=0 >> < >> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7Cdd2d1fd4495844536a9d08d75b7741ae%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637078440071742110&sdata=zOA%2FU%2BXBiwodptKThzDDrkWHE1%2FblZzz5EcJs%2BV23o4%3D&reserved=0 >> >* >> >> >> > > -- > > Piotr Zarzycki > > Patreon: *https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7Cdd2d1fd4495844536a9d08d75b7741ae%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637078440071742110&sdata=zOA%2FU%2BXBiwodptKThzDDrkWHE1%2FblZzz5EcJs%2BV23o4%3D&reserved=0 > <https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7Cdd2d1fd4495844536a9d08d75b7741ae%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637078440071752100&sdata=f1s%2B%2FTdASFpGNF8cGbxRyzjhs7Y1YTKHu2faQ1%2B1ikk%3D&reserved=0>* > -- Piotr Zarzycki Patreon: *https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7Cdd2d1fd4495844536a9d08d75b7741ae%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637078440071752100&sdata=f1s%2B%2FTdASFpGNF8cGbxRyzjhs7Y1YTKHu2faQ1%2B1ikk%3D&reserved=0 <https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7Cdd2d1fd4495844536a9d08d75b7741ae%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637078440071752100&sdata=f1s%2B%2FTdASFpGNF8cGbxRyzjhs7Y1YTKHu2faQ1%2B1ikk%3D&reserved=0>*
