CarlosRuiz: full meeting

2012-05-02T16:13:24Z

full meeting

New page

⇐
[[IDempiere|Table of Contents]] |
[[IDempiere/Full Meeting Minutes|Full Meeting Minutes]] |
Full Meeting 2012-05-02


'''''Nicolas__''''': Hello ! 
'''''CarlosRuiz''''': Good day! 
'''''tbayen_''''': Hi all! 
'''''mzuniga_ergio''''': Good day everybody :-) 
'''''CarlosRuiz''''': I'm reviewing open issues -> http://jira.idempiere.com/secure/IssueNavigator.jspa?requestId=10100 
'''''CarlosRuiz''''': please let me know if you want any specific to review 
'''''Nicolas__''''': i'd be interested in 245 (Strange behaviour for mandatory Date field which get the first focus ) ; i used a workaround but i think this bug has to be removed 
'''''CarlosRuiz''''': ok 
'''''Nicolas__''''': thanks 
'''''CarlosRuiz''''': Nicolas__, what do you mean with "click on Report Date and click on 'tab' ; calendar show ; select a date (double click)" - I can't reproduce the problem 
'''''Nicolas__''''': 'tab' means tabulation on keyboard 
'''''Nicolas__''''': just over caps lock 
'''''Nicolas__''''': don't know how it is in English 
'''''CarlosRuiz''''': yep - that's correct name 
'''''CarlosRuiz''''': I can't reproduce it - nothing strange happening 
'''''CarlosRuiz''''': the date is filled with the default and I can change it 
'''''CarlosRuiz''''': ah - wait - default logic 
'''''CarlosRuiz''''': I didn't do that part 
'''''Nicolas__''''': maybe it could be related 
'''''CarlosRuiz''''': just on swing - zk is fine 
'''''Nicolas__''''': yes 
'''''CarlosRuiz''''': and the calendar is opened automatically - no need to navigate 
'''''Nicolas__''''': yes 
'''''Nicolas__''''': VDate line 514 
'''''Nicolas__''''': because date is mandatory 
'''''Nicolas__''''': and wondering if it is a good thing that calendar opens automatically.... 
'''''Nicolas__''''': For instance, i'd prefer to write dates using keyboard. 
'''''CarlosRuiz''''': seems it was introduced here http://adempiere.svn.sourceforge.net/viewvc/adempiere/trunk/client/src/org/compiere/grid/ed/VDate.java?r1=1579&r2=1578&pathrev=1579 
'''''Nicolas__''''': so, don't run in debug mode, but ts would be null ?? 
'''''Nicolas__''''': that's why m_oldText value is selected (and is null) 
'''''CarlosRuiz''''': ok - Nicolas__ the issue is according to that code you cannot navigate out of a mandatory date field leaving it empty 
'''''CarlosRuiz''''': in zkwebui you can do that 
'''''Nicolas__''''': ok 
'''''CarlosRuiz''''': I tested dropping that part and it works 
'''''CarlosRuiz''''': I mean - focusLost just do this at the end: 
'''''CarlosRuiz''''': Object value = getValue(); 
'''''CarlosRuiz''''': setValue(value); 
'''''CarlosRuiz''''': what do you think about that solution? it put swing in sync with zk - not auto-opening calendar 
'''''Nicolas__''''': i like it 
'''''Nicolas__''''': +1 for me :-) 
'''''CarlosRuiz''''': it reopens this bug -> http://sourceforge.net/tracker/index.php?func=detail&aid=1660595&group_id=176962&atid=879332 
'''''Nicolas__''''': so the best solution would be to set a correct value (the one which is selected by calendar) 
'''''CarlosRuiz''''': testing ... 
'''''CarlosRuiz''''': Nicolas__, that leads me to a different error on GridController :-) 
'''''Nicolas__''''': great ! 
'''''Nicolas__''''': what kind of error ? 
'''''CarlosRuiz''''': the error described on sourceforge - when you copy a non-date string on a date field 
'''''CarlosRuiz''''': but such error is on GridController - not in VDate 
'''''CarlosRuiz''''': ready Nicolas__ , posted comment and patch on ticket 
'''''Nicolas__''''': ok, seen ; i will do some test and give feedback 
'''''Nicolas__''''': thanks 
'''''Nicolas__''''': i have to go, bye bye 
'''''CarlosRuiz''''': thanks Nicolas__, bye 
'''''CIA-126''''': iDempiere: globalqss * f27f1c9f736e r7271 /org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/component/AbstractADTab.java: 
'''''CIA-126''''': iDempiere: Rollback https://bitbucket.org/idempiere/idempiere/changeset/3b1e1c8a53a4 
'''''CIA-126''''': iDempiere: IDEMPIERE-31 Context problem when parent column can be in multiple tabs 
'''''buildmaster''''': Project iDempiere build #183: SUCCESS in 17 min: http://jenkins.idempiere.com/job/iDempiere/183/ 
'''''buildmaster''''': globalqss: Rollback https://bitbucket.org/idempiere/idempiere/changeset/3b1e1c8a53a4 
'''''buildmaster''''': IDEMPIERE-31 Context problem when parent column can be in multiple tabs 
'''''tbayen_''''': Bye!

IDempiere/FullMeeting20120502 - Revision history

CarlosRuiz: full meeting