IDempiere/FullMeeting20131127 - Revision history

CarlosRuiz: drop JIRA notifications from log

2013-11-27T16:40:17Z

drop JIRA notifications from log

CarlosRuiz: full meeting

2013-11-27T16:39:56Z

full meeting

New page

⇐
[[IDempiere|Table of Contents]] |
[[IDempiere/Full Meeting Minutes|Full Meeting Minutes]] |
Full Meeting 2013-11-27


'''''red1''''': Hola CarlosRuiz and everyone else. I am in Thailand.. where it is Yellow Shirts vs Red Shirts protest going on 
'''''CarlosRuiz''''': Hi red1 
'''''red1''''': I am asking the Thailand team to give their traditional greetings to CarlosRuiz 
'''''tbayen''''': As a general experience red shirts die early... 
'''''tbayen''''': http://en.wikipedia.org/wiki/Redshirt_%28character%29 
'''''pang''''': CarlosRuiz: Sawudee ka ^__^ 
'''''red1''''': there are two pretty Thai ladies here: Moodang and pang :D 
'''''red1''''': their photo is in my Facebook 
'''''Moodang''''': CarlosRuiz: Hi, How are you? 
'''''CarlosRuiz''''': tbayen, I received yesterday KitKat in my nexus 4 - running fine 
'''''CarlosRuiz''''': Hi pang, Moodang - everything going fine here 
'''''red1''''': I upgraded my Nexus 4 and it crashed now and then 
'''''red1''''': Android version 
'''''tbayen''''': :-) CarlosRuiz , yes I received it yesterday too. Seems we are in the same update cycle. It feels like a new phone. :-) 
'''''CarlosRuiz''''': yep - sounds like I was one of the lucky - read some comments about KitKat failing on nexus4 
'''''red1''''': Its time to get a new Nexus 
'''''red1''''': when is our next Krefeld conference? :D 
'''''red1''''': anyway i updated my iDempiere source and it crashes too 
'''''red1''''': ah.. i have to use mquery instead 
'''''Moodang''''': CarlosRuiz: Have you ever come to Thailand? 
'''''tbayen''''': Don't give red1 a new Nexus! He crashes everything! 
'''''red1''''': mspec i mean 
'''''red1''''': OK, i will hold the next iDempiere world conference in Bangkok 
'''''red1''''': and we will give tbayen a better gift 
'''''CarlosRuiz''''': Hi everybody 
'''''CarlosRuiz''''': Norbert proposed a discussion for the meeting but he has not arrived 
'''''nmicoud_''''': Bonjour 
'''''tbayen''''': Daarestiet! 
'''''nmicoud_''''': Carlos, WDYT of https://groups.google.com/forum/?fromgroups#!topic/idempiere/aspBJWajtBs ? 
'''''nmicoud_''''': should we open a jira ticket and attach regularly migration scripts to update column length ? 
'''''red1''''': ah.. now i am hitting thousands of errors 
'''''red1''''': cyclic dependency... nice 
'''''CarlosRuiz''''': yep - sounds good 
'''''nmicoud_''''': ok ; will do this then ; maybe wait 2 or 3 days , and then create the 1st script 
'''''nmicoud_''''': About https://idempiere.atlassian.net/browse/IDEMPIERE-1461 ; do you know if hengsin does something ? 
'''''nmicoud_''''': he suggest a new approach 
'''''nmicoud_''''': but i don't know who can develop it 
'''''nmicoud_''''': not sure i'm the good person as it's quite complex i think 
'''''tbayen''''': Do you agree that setting the length of AD_PInstance_Para.P_String to a length as big as 5000 does no harm? 
'''''CarlosRuiz''''': the only collateral I see for that is printing such string in the PDF header - maybe you could try to trim it and add "..." at the end if is longer than NNN chars 
'''''CarlosRuiz''''': nmicoud_, I don't get completely the idea from Heng Sin on IDEMPIERE-1461 
'''''CarlosRuiz''''': don't see the value added splitting the zoom logic to several tables 
'''''nmicoud_''''': if i understand well : AD_ZoomCondition will be used when recordID > 0 (conditional) and new table used for handling context variables (@#AD_Client_ID@=10000...). And empty record will used this 
'''''nmicoud_''''': i think it's easier to do 
'''''nmicoud_''''': i'd prefer to have everything in the same place but hengsin think it would be better this way (look his comment 11/12) 
'''''CarlosRuiz''''': as I see the context logic would be applied for recordid=0 - and the context+SQL applied for recordid>0 - and you can have several conditions defined 
'''''nmicoud_''''': yes 
'''''CarlosRuiz''''': and they're applied ordered by seqno 
'''''nmicoud_''''': yes, and the 1st correct is the winner 
'''''aguerra''''': Hi E very boda!!! 
'''''aguerra''''': Hi everybody 
'''''nmicoud_''''': hi 
'''''CarlosRuiz''''': Hi Alejandro 
'''''aguerra''''': Hi, nmicoud, @CarlosRuiz 
'''''red1_''''': Hola aguerra 
'''''aguerra''''': Hola mr @red1 
'''''Not-002''''': [IDEMPIERE] deepak created IDEMPIERE-1572 Last Run and Saved parameter for process 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1572 
'''''CarlosRuiz''''': nmicoud_, I'll try a POC for IDEMPIERE-1461 - let me check 
'''''CarlosRuiz''''': it doesn't sound too hard 
'''''nmicoud_''''': :) thanks 
'''''Not-002''''': [IDEMPIERE] deepak updated IDEMPIERE-1572 Attachment set to "LastRun.png" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1572 
'''''CarlosRuiz''''': Deepak_, 1572 looks good - is it based on AD_PInstancePara? 
'''''Deepak_''''': yes 
'''''Deepak_''''': CarlosRuiz, we have done for zk but yet working on swing in idempiere 
'''''tbayen''''': Fine! I will test the swing version if you like 
'''''CarlosRuiz''''': excellent news 
'''''CarlosRuiz''''': I can test the zk :-) 
'''''nmicoud_''''': About https://idempiere.atlassian.net/browse/IDEMPIERE-1521 (ability to delete a view within AD), i wanted to execute the DROP... in the afterDelete of Mtable (this will prevent from adding another process) ; WDYT ? 
'''''CarlosRuiz''''': I think the button can be better - as you implemented it for index 
'''''CarlosRuiz''''': i.e. if you want to change a view to become a table 
'''''nmicoud_''''': right ; will do it this way then 
'''''tbayen''''': Has someone looked into IDEMPIERE-1490? It is a big performance problem if you work with bigger tables. 
'''''CarlosRuiz''''': deleting a view can throw errors in postgres when is used by other objects - in oracle it doesn't just mark the referenced objects as invalid 
'''''Not-002''''': [IDEMPIERE] nmicoud updated IDEMPIERE-1521 assignee set to "Nicolas Micoud" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1521 
'''''CarlosRuiz''''': I haven't yet tbayen 
'''''Deepak_''''': tbayen and CarlosRuiz, I will update you both once patch is added on Jira 
'''''tbayen''''': For me the patch in IDEMPIERE-1490 works and is logic. I am just a bit confused how this bug could ever be introduced. That's why I ask for review. 
'''''tbayen''''': Deepak_, thanks! 
'''''tbayen''''': CarlosRuiz, if we are in views, we can at the same time look into IDEMPIERE-659. I use it for a long time and hope it is stable. I can create one single patch for that if you like this better. 
'''''norbertbede''''': hola 
'''''CarlosRuiz''''': Hi norbertbede 
'''''norbertbede''''': what is main topic? or can i nominate yesterdays Info Window issue ? 
'''''CarlosRuiz''''': not main topic at this moment - yes - we can check about Info window if you want 
'''''norbertbede''''': yes 
'''''norbertbede''''': it is issue https://idempiere.atlassian.net/browse/IDEMPIERE-1568 
'''''norbertbede''''': so in my last comment i describe how we can implement solution which is de-facto standard in CRM systems including mass update, etc... 
'''''norbertbede''''': would be nice if others should ask, or suggest, or reject ideas 
'''''norbertbede''''': original idea was missing WMS component smartbrowse, but it should be nice way implemented into Info windows - and much more... 
'''''CarlosRuiz''''': as I wrote a draft idea would be just to add a process_id to the info window 
'''''CarlosRuiz''''': I mean - the info windows already have the sections 1 and 2 of what you described 
'''''CarlosRuiz''''': the section 3 is process parameters and a button 
'''''CarlosRuiz''''': so - adding the process_id could make the trick - the implementation of info windows would add the button and the process parameters to the info window and behaves accordingly 
'''''norbertbede''''': ok. so that is my idea so i allready describe my requirements. anybody others ? Just outstanding question who and how we can implement it. 
'''''norbertbede''''': i should help to test, make documentation, and test in WMS migration 
'''''norbertbede''''': which is next step after this issue 
'''''Not-002''''': [IDEMPIERE] dantam updated IDEMPIERE-1396 
'''''Not-002''''': [IDEMPIERE] I looked into this and the error message is a safeguard against matching information becoming corrupt which is good. I've made a solution to this particular problem and will try to create a working pull request for it. 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1396 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1461 Attachment set to "IDEMPIERE-1461.patch" 
'''''Not-002''''': [IDEMPIERE] Attaching IDEMPIERE-1461.patch This is working this way: The zoom condition records are evaluated ordered by seqno, firstly the ZoomLogic is evaluated, if the logic returns true then the WhereClause is evaluated on the target record - if there is no WhereClause then it applies for all records including New. Attaching also the 2pack for three zoom conditions that I used for tests on AD_User table. Regards, Carlos Ruiz 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1461 
'''''CarlosRuiz''''': nmicoud_, patch for IDEMPIERE-1461 uploaded 
'''''CarlosRuiz''''': the only drawback is that on new it's not opening the zoom window in "New" mode 
'''''CarlosRuiz''''': ah - BTW patch is for release-2.0 
'''''nmicoud_''''': ok, will check that ; thanks 
'''''nmicoud_''''': did you test it with bp location ? eg: you're on an order ; select a bpartner which has no location 
'''''nmicoud_''''': you click on location ; does it open the correct tab ? 
'''''CarlosRuiz''''': no, just tested with user from SO 
'''''nmicoud_''''': ok, will it and give feedback 
'''''ocurieles''''': Hi everybody !!! 
'''''CarlosRuiz''''': Hola Orlando 
'''''ocurieles''''': the upgrade script for db is working @CarlosRuiz ?? 
'''''ocurieles''''': i wanna put the url in the wiki 
'''''CarlosRuiz''''': ocurieles, there are three 
'''''CarlosRuiz''''': the three are working 
'''''ocurieles''''': http://bitbucket.org/CarlosRuiz_globalqss/idempiere-stuff/raw/tip/script_to_sync_db/syncApplied_1.0c.sh 
'''''ocurieles''''': this ? is in the wiki ? 
'''''Not-002''''': [iDempiere] CarlosRuiz_globalqss pushed 2 commits to development [+2/-0/±5] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-002''''': [iDempiere] globalqss 512ef08 - hg merge release-1.0c (merge release1 into release2) 
'''''Not-002''''': [iDempiere] globalqss 6b138eb - hg merge release-2.0 (merge release2 into development) 
'''''CarlosRuiz''''': that is for version 1, there is another for version 2 and another if you keep playing with development branch 
'''''ocurieles''''': and where is the link ? 
'''''CarlosRuiz''''': same folder 
'''''ocurieles''''': i was updated with migration folder of the source code using dos2unix 
'''''Not-002''''': [IDEMPIERE] dantam created IDEMPIERE-1573 Currency variance on payments with foreign currency from bank account isn't calculated 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1573 
'''''Not-002''''': [IDEMPIERE] dantam updated IDEMPIERE-1573 Attachment set to "IDempiere-1573.png" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1573 
'''''ocurieles''''': http://bitbucket.org/CarlosRuiz_globalqss/idempiere-stuff/raw/tip/script_to_sync_db/syncApplied_2.0.sh 
'''''ocurieles''''': this work too 
'''''Not-002''''': [IDEMPIERE] nmicoud updated IDEMPIERE-1521 Attachment set to "IDEMPIERE-1521.patch" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1521 
'''''Not-002''''': [IDEMPIERE] nmicoud updated IDEMPIERE-1521 assignee set to "Carlos Antonio Ruiz Gomez" 
'''''Not-002''''': [IDEMPIERE] Hi, Please find a patch that add a button on AD_Table to drop the view. Regards, Nicolas 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1521 
'''''Not-002''''': [IDEMPIERE] nmicoud updated IDEMPIERE-1521 labels set to "+Patch" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1521 
'''''Not-002''''': [IDEMPIERE] nmicoud updated IDEMPIERE-1521 status set to "Peer Review Queue" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1521 
'''''Not-002''''': [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.0 [+2/-0/±0] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-002''''': [iDempiere] globalqss cb7dcb1 - IDEMPIERE-594 Improve positioning on windows / fix column tab 
'''''CarlosRuiz''''': gtg guys - thanks for the meeting

@@ Line 63: / Line 63: @@
 '''''red1_''''': Hola aguerra<br>
 '''''aguerra''''': Hola mr @red1<br>
 '''''CarlosRuiz''''': nmicoud_, I'll try a POC for IDEMPIERE-1461 - let me check<br>
 '''''CarlosRuiz''''': it doesn't sound too hard<br>
 '''''nmicoud_''''': :) thanks<br>
 '''''CarlosRuiz''''': Deepak_, 1572 looks good - is it based on AD_PInstancePara?<br>
 '''''Deepak_''''': yes<br>
@@ Line 82: / Line 78: @@
 '''''tbayen''''': Has someone looked into IDEMPIERE-1490? It is a big performance problem if you work with bigger tables.<br>
 '''''CarlosRuiz''''': deleting a view can throw errors in postgres when is used by other objects - in oracle it doesn't just mark the referenced objects as invalid<br>
 '''''CarlosRuiz''''': I haven't yet tbayen<br>
 '''''Deepak_''''': tbayen and CarlosRuiz, I will update you both once patch is added on Jira<br>
@@ Line 105: / Line 99: @@
 '''''norbertbede''''': i should help to test, make documentation, and test in WMS migration<br>
 '''''norbertbede''''': which is next step after this issue<br>
 '''''CarlosRuiz''''': nmicoud_, patch for IDEMPIERE-1461 uploaded<br>
 '''''CarlosRuiz''''': the only drawback is that on new it's not opening the zoom window in "New" mode<br>
@@ Line 127: / Line 115: @@
 '''''ocurieles''''': http://bitbucket.org/CarlosRuiz_globalqss/idempiere-stuff/raw/tip/script_to_sync_db/syncApplied_1.0c.sh<br>
 '''''ocurieles''''': this ? is in the wiki ?<br>
 '''''CarlosRuiz''''': that is for version 1, there is another for version 2 and another if you keep playing with development branch<br>
 '''''ocurieles''''': and where is the link ?<br>
 '''''CarlosRuiz''''': same folder<br>
 '''''ocurieles''''': i was updated with migration folder of the source code using dos2unix<br>
 '''''ocurieles''''': http://bitbucket.org/CarlosRuiz_globalqss/idempiere-stuff/raw/tip/script_to_sync_db/syncApplied_2.0.sh<br>
 '''''ocurieles''''': this work too<br>
 '''''CarlosRuiz''''': gtg guys - thanks for the meeting<br>