IDempiere/FullMeeting20140514

⇐ Table of Contents | Full Meeting Minutes | Full Meeting 2014-05-14

CarlosRuiz: Good morning
nmicoud: Bonjour
CarlosRuiz: will be checking queue
nmicoud: any chance to integrate https://idempiere.atlassian.net/browse/IDEMPIERE-1899 ?
CarlosRuiz: sure let me check it today
nmicoud: thanks
Deepak_: Good Morning/ Noon Everybody
Deepak_: Carlos, Do we have iDempiere architecture diagram?
CarlosRuiz: database?
Deepak_: No
Deepak_: Like system architecture document?
Not-6d9: [IDEMPIERE] tsvikruha updated IDEMPIERE-1334 Attachment set to "IDEMPIERE-1334-fixed_info_panel.patch"
Not-6d9: [IDEMPIERE] Hi, i attached patch for InfoPanel.java class. Changes in patch: 1. At the end of method you has to disable all buttons, not only one from the process was called otherwise you can get java.null.pointer.exception. 2. Do not show popup dialog with empty return message or after run process (empty message or with text '&Report' is little bit weird) As Next, i suggest next other changes: 1. When is set process on Info Window by original approac
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1334
CarlosRuiz: I don't think so - I've seen several trials to document that - but none complete
hieplq: @CarlosRuiz, I sugest make new ticket for add more comment code, at least head of class and function
CarlosRuiz: yep - that sounds fine
CarlosRuiz: I remember reading about some agile practices against comments - not saying that we need to apply it here :-)
Not-6d9: [IDEMPIERE] norbert.bede updated IDEMPIERE-1334 Attachment set to "infowindow-process-buttons.png"
Not-6d9: [IDEMPIERE] [~hieplq] rethink my original idea. each proces is 1 button. in some cases should be helpful. we have 3 options 1. dropdown and process button 2. each process 1 button 3. support both - behaviour defined in info window. (if technically possible then i vote for this) - means case 1 should be setup as option 1. case 2 as option 2.
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1334
norbertbede: hi all dear team
CarlosRuiz: Hi norbertbede
norbertbede: we worked with hiep and together on issue 1334 - Run one or more processes from Info Window
norbertbede: looks me we have base code and would be nice to say opinion - screenshot and code is attached
norbertbede: we need support on this to decide and finish
norbertbede: thnks
hieplq: hi norberbede.
hieplq: I think option 3 can with a responsive design
hieplq: list button show when have a 3,4 button (it's rely to text in button) when total with of all button > max, auto swith to dropdow
CarlosRuiz: reading ...
hieplq: @CarlosRuiz, wdyt about IDEMPIERE-1334, 1. add support multi process button, 2. remove button of red, combine it to multi button, 3. add default print button
CarlosRuiz: yep - reading that one
CarlosRuiz: option 3 could be also to add a IsButton flag
CarlosRuiz: if not button - then is a dropdown
CarlosRuiz: the imageURL just make sense for button?
hieplq: yes.
CarlosRuiz: as a user I think I would prefer buttons - as implementor I would prefer to have the option for dropdown too :-)
hieplq: how to when too much button?
Deepak_: I thing adding multiple button make more sense
hieplq: yes. i also like button, but must resolve case have too much button
CarlosRuiz: you just keep adding buttons to the toolbar - it would jump to next line?
norbertbede: we talk about compact (dropdown) and comfortable version. i
hieplq: @CarlosRuiz. never test, just think about this case. will test it.
hieplq: but muilti line is not nice :)
CarlosRuiz: yep, that's when configurable dropdown makes sense
norbertbede: because that i why i vote for make default dropdown, make configurable isbutton
norbertbede: dropdown version not yet coded.
hieplq: it's not hard. will soon make it affter meeting
norbertbede: ok. thnks
CarlosRuiz: hieplq, and we would need to migrate the actual processID on master to the detail, and hide or inactivate the processID column on master
hieplq: yes. will do.
CarlosRuiz: another suggestion - when filling the dropdown list of processes (or list of buttons) - you need to check for role access
CarlosRuiz: if the role doesn't have access better not to show the button just to say later "you cannot execute it"
hieplq: ah. yes like this.
hieplq: @CarlosRuiz, how about default print button.
hieplq: i remeber hear it in a meeting
CarlosRuiz: what's that?
CarlosRuiz: on info window?
hieplq: yes.
hieplq: have a default print format for all info window. it print as excel
CarlosRuiz: like the account info
CarlosRuiz: that sounds good - just check the role has export permission
CarlosRuiz: now, wondering if you can set a report instead of a process in the button list
hieplq: report is more diffirence from process?
CarlosRuiz: in dictionary is the same
CarlosRuiz: reports are processes providing an output
hieplq: with output, I can't call it from button?
CarlosRuiz: not sure how would that work
CarlosRuiz: just a draft idea - no need to implement it at this stage
hieplq: ah yes. just idea. will discover process of report late
Deepak_: Yes, default print format. Also configurable report
Deepak_: Some time need to have multiple print format for same info view
hieplq: same print button in adwindow :)
Deepak_: yes, but making user friendly look
Deepak_: For example we can have easy access to print pick ticket, shipping label and invoice from shipment info window
Not-6d9: [IDEMPIERE] hieplq created IDEMPIERE-1947 add comment code
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1947
norbertbede: i can imagine to show processess and print formats in same dropdown. we need a separator between and add prefix: process/report - change process button name to "RUN/start"
norbertbede: then we save place make it compact and improve functionality lot
hieplq: @CarlosRuiz. other ticket i want ask. https://idempiere.atlassian.net/browse/IDEMPIERE-1540
hieplq: I think autocomplete shoud replace default combobox in case combobox have a lot item
CarlosRuiz: there is already a flag AD_Column.IsAutoComplete
CarlosRuiz: is just enabled for strings AFAIR
CarlosRuiz: you could use it also for this purpose
hieplq: ah. i will implement autocomplete and leave for implementor decide.
CarlosRuiz: nmicoud, reviewing IDEMPIERE-1899
nmicoud: ok, thanks
CarlosRuiz: the method setContent is not used - can I drop it? or you need it?
Not-6d9: [IDEMPIERE] norbert.bede updated IDEMPIERE-1945 summary set to "Tree Maintenance does not correctly display fields"
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1945
nmicoud: let me check, do not remember
nmicoud: i've create a new setContentSignature method to keep the old one, but just for test purpose
nmicoud: i think we could remove the 'old' setContent
nmicoud: and rename setContentSignature =>setContent
CarlosRuiz: ok
Not-6d9: [IDEMPIERE] nmicoud created IDEMPIERE-1948 Tree : Arrow in front of summary node is not consistent
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1948
Not-6d9: [IDEMPIERE] hieplq updated IDEMPIERE-1924 status set to "Closed" -resolution set to "Duplicate"
Not-6d9: [IDEMPIERE] will fix in IDEMPIERE-1334
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1924
norbertbede: @carlos take a look please to this. https://idempiere.atlassian.net/browse/IDEMPIERE-1792
norbertbede: we found default logic field was removed from info window. why. we cant get answer why how to set default value
norbertbede: eg. if open info window i want show dropdown active records = default
norbertbede: as in standard window
CarlosRuiz: have you tried with an advanced role?
norbertbede: hm not. but i will try it now mmt
norbertbede: i login as SuperUser@System.*/System Administrator opening Info window. default logic field mssing
CarlosRuiz: let me check in demo ...
norbertbede: this role has advanced role flag true
norbertbede: thanks
norbertbede: my developer report this too. believe its not our mistake...
CarlosRuiz: ah sorry - I thought the field was there before - but you say it has not been implemented
CarlosRuiz: yes - that can be helpful - as well as implementing preferences for those fields
norbertbede: we have in database. but that was migrated from adempiere
norbertbede: then we thought was removed
norbertbede: im crying now why im consultant cant coding.. :)
Not-6d9: [IDEMPIERE] norbert.bede updated IDEMPIERE-1792
Not-6d9: [IDEMPIERE] [#idempiere Meeting in Progress every Wednesday 13:00 UTC (Project landing page: www.idempiere.com)] [17:19] <norbertbede> @carlos take a look please to this. https://idempiere.atlassian.net/browse/IDEMPIERE-1792 [17:19] <norbertbede> we found default logic field was removed from info window. why. we cant get answer why how to set default value [17:20] <norbertbede> eg. if open info window i want show dropdown active records = default [17:
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1792
Not-6d9: [IDEMPIERE] norbert.bede created IDEMPIERE-1949 Replace 4 standard Forms with new Info Window
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1949
CarlosRuiz: nmicoud, the add text automatically is not happening
Not-6d9: [IDEMPIERE] norbert.bede updated IDEMPIERE-1815
Not-6d9: [IDEMPIERE] issue 1788 will resolve this request
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1815
nmicoud: ah ? with patch v2 ?
CarlosRuiz: I see the code there - let me debug
CarlosRuiz: instead of new MUser(Env.getCtx(), Env.getAD_User_ID(Env.getCtx()), null)
CarlosRuiz: is better to use MUser.get
CarlosRuiz: that's not the issue - just a recommendation
CarlosRuiz: the problem was the tester :-D
nmicoud: héhé
nmicoud: first i used the cache
nmicoud: but then i remove it
Not-6d9: [IDEMPIERE] cboecking updated IDEMPIERE-1815
Not-6d9: [IDEMPIERE] Hi Norbert, does 1788 apply to the Generate Shipments Manual form as well?
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1815
CarlosRuiz: nmicoud, added one line and now it can parse the Name of the user
nmicoud: good
Not-6d9: [IDEMPIERE] norbert.bede updated IDEMPIERE-1815
Not-6d9: [IDEMPIERE] issue 1949 is part of solution
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1815
norbertbede: @carlos - simple but strategic decision required - https://idempiere.atlassian.net/browse/IDEMPIERE-1941
CarlosRuiz: nmicoud, great - received an email from Clark Kent
nmicoud: tada !
CarlosRuiz: uh oh
CarlosRuiz: but I have one problem
CarlosRuiz: two problems indeed
nmicoud: ah...
CarlosRuiz: I'll upload my modified patch - and comment there
CarlosRuiz: 1 - the other attachments are lost
CarlosRuiz: 2 - user would need to write his/her text in html notation
CarlosRuiz: all the non-html formatting from the user (i.e. newlines) is lost
CarlosRuiz: I'll upload a patch as I did some fixes
CarlosRuiz: BTW - diff --git
CarlosRuiz: to include binaries in the patch
nmicoud: ok will try it (but using windows)
Not-6d9: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1899 Attachment set to "IDEMPIERE-1899_v3.patch"
Not-6d9: [IDEMPIERE] Uploading v3 modified patch (including binary): problems found: 1 - the other attachments are lost, i.e. when calling the feedback email to support button the screenshot and log are not available in the received message Samely when sending a report the pdf is missing 2 - user would need to write his/her text in html notation, it means all the non-html formatting from the user (i.e. newlines) is lost I think it becomes necessary to have an
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1899
nmicoud: for the html notation, perhaps we should integrate the ckEditor in WEmailEditor ?
CarlosRuiz: norbertbede, about IDEMPIERE-1941, what if you want to "inactivate" temporarily a locator (i.e. to avoid people using it)
CarlosRuiz: yep nmicoud - the string editor already has HTML capabilities in any string field
nmicoud: i means, you won't have to swith to the html mode
nmicoud: when opening, you will have the toolbar and besides the text
CarlosRuiz: I mean - what if somebody prefer to send plain text mail?
nmicoud: he won't :)
nmicoud: i see what you mean
nmicoud: so, maybe changing the current text field editor with a ckEditor ?
nmicoud: and right click / html to switch mode ?
CarlosRuiz: let me check if hengsin is available
nmicoud: and http://www.zkoss.org/zkmail/ ? or am i dreaming ?
norbertbede: @Carlos about IDEMPIERE-1941, we should implement dropdown - locator status (active, error, damaged...etc). i can design that.
norbertbede: main reason to make compactible with a big warehouse setup. and avoid picking sequences errors in layout
proSENSE: Is it possible, to configure BP or any other import, to import corretly latvian language... now its importing AÌENTS.LV as A?ENTS.LV
CarlosRuiz: proSENSE, must be an issue of the encoding you select
proSENSE: UTF-8
CarlosRuiz: nmicoud, in principle hengsin agreed that we can deprecate plain text email - and use just html - that can make the integration of the html editor easier
nmicoud: ah great!
CarlosRuiz: if is not too hard - maybe you can implement what you said
nmicoud: means that every mail sent will be HTML ?
CarlosRuiz: a toggle or right click for plain text
CarlosRuiz: but if is too hard - we can deprecate plain text
CarlosRuiz: and force html email always
nmicoud: will try it
nmicoud: but perhaps this would have to be done before 1899 ?
Not-6d9: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1899 status set to "Open" -assignee set to "Nicolas Micoud"
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1899
proSENSE: when there is a processing sales invoice - is it possible to add "service entry" or anything else as free txt? Or its needed to be entered in stock to sale?
CarlosRuiz: norbertbede, IMHO the active flag is useful in that case - to filter out things temporarily
CarlosRuiz: i.e. we don't have a way to "isolate" a portion of the warehouse during an inventory count (solved if we implement some perpetual inventory approach)
CarlosRuiz: but until those features are implemented I think allowing inactivate a locator is useful
norbertbede: ok. its ok. i will try to design this locator status proposal in a new issue
norbertbede: maybe system configurator should give us trust and make compactive until that
norbertbede: we found the issue when stockeeper deactivate 50 locators and system skips those locators on picking = make real trouble for company. in KNOW that is a specific area...
norbertbede: they scrap FIFO..
CarlosRuiz: what unpredictable behavior do you foresee?
CarlosRuiz: ah I see
CarlosRuiz: remembered a joke "a well documented bug can be a feature" :-D
CarlosRuiz: but seriously - I think the case is valid
CarlosRuiz: what if you want temporarily the system to skip a locator on picking? then that's the way
CarlosRuiz: the issue can be maybe to restrict the active flag to that user - and allow it just for a manager
norbertbede: main problem happening: when warehouse layout changing frequently, we found that after 1 year. on weekly basis for hundred of locations.
norbertbede: that shows, in extreme use cases cause issue
norbertbede: with active gflag. the problem is nobody know why a locator was deactivated. no purpose/reason
norbertbede: i dont want to push to base wms features for now
norbertbede: but we found to denied warehouse and locator deactivation when qty <>0 reasonable
norbertbede: hard to answer
norbertbede: this
norbertbede: why
norbertbede: the compromisse should be as you wrote - make role dependent that
norbertbede: agree.
norbertbede: for the future we can apply later locator statues, not importnt now.
Not-6d9: [iDempiere] CarlosRuiz_globalqss pushed 1 commit to development [+4/-1/±9] https://bitbucket.org/idempiere/idempiere/commits/
Not-6d9: [iDempiere] hengsin d927f81 - 1003862 Barcode type UPC A is needed for print formats going to Amazon. Added UPCA support using barcode4j. Use barcode4j instead of barbecue as I can't get decent output from barbecue, barbecue have the underline bug and barbecue is a dead project. Should completely drop the use of barbecue in future.
Not-6d9: [IDEMPIERE] carlosruiz_globalqss created IDEMPIERE-1950 Product Info Reservation not showing if Warehouse > Reservation Locator not populated
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1950
Not-6d9: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1950 status set to "Resolved" -assignee set to "Heng Sin Low" -resolution set to "Fixed"
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1950
Not-6d9: [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.0 [+2/-0/±4] https://bitbucket.org/idempiere/idempiere/commits/
Not-6d9: [iDempiere] hengsin d66ef17 - 1003965 IDEMPIERE-1950 Product Info Reservation not showing if Warehouse > Reservation Locator not populated.
Not-6d9: [IDEMPIERE] carlosruiz_globalqss created IDEMPIERE-1951 Run report as a background job
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1951
Not-6d9: [iDempiere] CarlosRuiz_globalqss pushed 4 commits to development [+2/-0/±12] https://bitbucket.org/idempiere/idempiere/commits/
Not-6d9: [iDempiere] hengsin bd127ab - 1003754 Landed cost issue -- related to IDEMPIERE-1285. Fix landed cost posting.
Not-6d9: [iDempiere] elainetan 3705c28 - IDEMPIERE-1944 Reverse Issue with GL Journal and GL Journal Batch
Not-6d9: [iDempiere] hengsin 3938998 - 1003754 Landed cost issue -- related to IDEMPIERE-1285. Fix rounding issue. Fix wrong posting when estimated landed cost is more than actual landed cost.
Not-6d9: [iDempiere] globalqss 481fe26 - hg merge release-2.0 (merge release2 into development)
Not-6d9: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1944 status set to "Resolved" -resolution set to "Fixed"
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1944
Not-6d9: [iDempiere] CarlosRuiz_globalqss pushed 3 commits to development [+6/-0/±13] https://bitbucket.org/idempiere/idempiere/commits/
Not-6d9: [iDempiere] globalqss a02ba14 - 1003856 IDEMPIERE-1951 Integrate commit from hengsin -> Run report as a background job. Peer review implementation from Elaine.
Not-6d9: [iDempiere] hengsin 5ec3255 - 1003856 IDEMPIERE-1951 Run report as a background job. Fix bug discover during testing.
Not-6d9: [iDempiere] hengsin 5977d10 - 1003856 IDEMPIERE-1951 Run report as a background job. Fix counts of background user jobs included scheduler jobs.
Not-6d9: [IDEMPIERE] carlosruiz_globalqss created IDEMPIERE-1952 Create ability to run reports in the scheduler using context variables for dates
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1952
Not-6d9: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1951 status set to "Resolved" -resolution set to "Fixed"
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1951
Not-6d9: [iDempiere] CarlosRuiz_globalqss pushed 2 commits to development [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/
Not-6d9: [iDempiere] hengsin 8a02512 - 1003949 IDEMPIERE-1952 Create ability to run reports in the scheduler using context variables for dates. Implement context variable support for scheduler name.
Not-6d9: [iDempiere] hengsin 2676072 - 1003949 IDEMPIERE-1952 Create ability to run reports in the scheduler using context variables for dates. Add plus and minus arithmetic support for @#Date@ context variable.
Not-6d9: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1952 status set to "Resolved" -assignee set to "Heng Sin Low" -resolution set to "Fixed"
Not-6d9: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1952