IDempiere/FullMeeting20120502

⇐ Table of Contents | 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!