IDempiere/FullMeeting20140903

⇐ Table of Contents | Full Meeting Minutes | Full Meeting 2014-09-03

Deepak: Good Morning All
Deepak: CarlosRuiz, can you review IDEMPIERE-1687. It is in review queue since quite long time
CarlosRuiz: Good morning
Deepak: Good Morning Carlos
JanThielemann: hi
CarlosRuiz: Deepak, at this moment I'm focused on r3 - I think that will be delayed for next release
CarlosRuiz: Hi JanThielemann
CarlosRuiz: JanThielemann, you were on vacations when we were playing with your android zerlina contacts
CarlosRuiz: thanks about that!
Deepak: ohk
CarlosRuiz: I just posted a couple of posts commenting at
CarlosRuiz: https://groups.google.com/d/msg/idempiere/tmCk4bAkym8/sg8wQU0JhYoJ
tbayen: Ah. Yes! Hi JanThielemann I forgot to mention that. We really liked your Android App. I want to build more thing on that base. Great work!
ocurieles_DCS: Hi... Good Morning
JanThielemann: thats great to hear :)
CarlosRuiz: Hi Orlando
ocurieles_DCS: @JanThieleman, congratulations, i'm checking your youtube videos and is a great work too...
ocurieles_DCS: the progressive search is configurable ? maybe some system configurator ?
CarlosRuiz: what do you mean with progressive search?
ocurieles_DCS: in the menu Search
ocurieles_DCS: the search is working letter by letter
ocurieles_DCS: is it configurable ?
CarlosRuiz: ah yes, work in progress - we are discussing that at ticket https://idempiere.atlassian.net/browse/IDEMPIERE-2050
ocurieles_DCS: Thanks
tbayen: In my development system I can't access the archive. And I swear it worked two months before. Is there anything changed in this area or is it my fault?
tbayen: When I press the archive toolbar button it says that there are documents but when I press one of the lines from the pulldown menu nothing happens. :-(
JanThielemann: how do i get an entry there?
CarlosRuiz: tbayen, reproducible on demo?
JanThielemann: ah, created a report for bpartner, saved it to archive and when i hit the button a new window archive viewer opens where i can see the report
JanThielemann: (on demo)
tbayen: Not yet tried. Before that I wanted to ask if someone worked an that. AS you dont' cry "look here!" I will try to reproduce it.
tbayen: Demo is 2.0, isn't it? My development system is quite actual (ca. 2 weeks ago).
CarlosRuiz: demo is dev
tbayen: Hmmm... if Jan tried it it must be my fault. ok. :-( I will debug it.
JanThielemann: your archive viewer does not open?
tbayen: Jan - No.
JanThielemann: any errors on the console?
ocurieles_DCS: i Tried in DEMO, in sales order can archive the format, but is not working for sales order report
CarlosRuiz: archiving sales order has been always tricky
tbayen: What is different with Sales Order?
CarlosRuiz: when you push the print button on a completed sales order it shows the related invoice instead of the sales order
CarlosRuiz: and if you archive it - it's archived related to the invoice
tbayen: My problem is reproducible in demo. Perhaps I am just too stupid.
CarlosRuiz: on sales order?
ocurieles_DCS: I tryed in Business Partner and this work too ...
tbayen: I log in as GardenAdmin. On Invoice 1000007 I printed and archived the document. When I press the archive toolbar button I get al line "Documents (1)". That opens a search window without any content in the search fields. How do I open the archive to see this printed Invoice?
tbayen: And no - in my own instance I get no message in the log.
JanThielemann: theres another tab
JanThielemann: inside the window
tbayen: ok JanThielemann , thanks! I AM too stupid. :-(
JanThielemann: :D
JanThielemann: i didn't saw it too at first
JanThielemann: don't worry
CarlosRuiz: ah!!!
CarlosRuiz: I noticed that recently
JanThielemann: i clicked the checkmark like 10 times and wondered why the window closed xD
CarlosRuiz: I think is a bug -> archive viewer is opening on query instead of view result
tbayen: I like to have a new JIRA category "not a bug but stupid users will do this wrong".
tbayen: JanThielemann, thanks that you also pressed the checkmark. You give me back some self-confidence. ;)
JanThielemann: it looked like you should press it xD
HideakiHagiwara: Thank you for CarlosRuiz to resolve IDEMPIERE-2170
CarlosRuiz: thanks to you HideakiHagiwara
CarlosRuiz: tbayen, I think that's a bug - the viewer must be opened on view result tab when coming from the button - and on the query tab when opened from the menu
tbayen: yep. I will file a ticket. :(
CarlosRuiz: AFAIR it used to work like that
tbayen: I think it worked. I worked with it some weeks ago. But I can not swear that I just pressed the tab every time.
tbayen: CarlosRuiz, I made a new patch for IDEMPIERE-2137 and tested it. Can you review it?
tbayen: Ah! You did your own private work. ;-)
tbayen: Fine. Thanks!
CarlosRuiz: preferred to make just a comment, in case somebody wants to use it again later - or in a plugin
CarlosRuiz: BTW - I thought this was related with the other finding - but is not
tbayen: Yep. As always your code is better than mine.
CarlosRuiz: when I imported the webservice access definition from GardenWorld to a new tenant
CarlosRuiz: it was exported using Data
CarlosRuiz: it imported wrongly
CarlosRuiz: the access definition on new tenant had reference to GardenAdmin
CarlosRuiz: I'll better open a ticket for that
tbayen: I still do not exactly understand it. So better open a ticket.
tbayen: ok. Closed this one - going to the next. Can you review IDEMPIERE-2140?
CarlosRuiz: ok - described there -> IDEMPIERE-2179
CarlosRuiz: tbayen, 2140 was not committed by hengsin?
tbayen: Ah. Yes! It was. Why is it not closed?
tbayen: :-)
tbayen: What about IDEMPIERE-2104? The patch might be useful not for many people but it it quite small and my customer really likes it.
tbayen: It gives the possibility to see the outgoing addresses very easy. And you can have a sort of gatekeeper who checks the outgoing mails.
CarlosRuiz: tbayen, did you test the possibility of having more than one recipient?
tbayen: no.
Deepak: tbayen, I liked IDEMPIERE-2104
tbayen: After looking into the code I would say it should work with more than one recipient.
CarlosRuiz: apparently must support several -> http://docs.oracle.com/javaee/1.4/api/javax/mail/internet/MimeMessage.html#setReplyTo(javax.mail.Address[])
CarlosRuiz: would be good to test if it works
tbayen: It does not work if there is already a ReplyTo address. But I think this case is so sophisitcated that I decided just to do nothing with the To address if that happens.
tbayen: ok. I will test it. Wait...
Nikita_: Hello, Carlos! I added support Single sign-on(SSO) in iDempiere for my project. Do you planning support SSO for iDempeire in future?
CarlosRuiz: sure Nikita_ - great if you can contribute that
Nikita_: Carlos. i will create topic in google forum and show how it work and describe existing problem
CarlosRuiz: thanks
CarlosRuiz: will be away for 1h - brb
blissmen: helloo
tbayen: Hi
Nikita_: hi
blissmen: hey Nikita_ I am Caleb but my friends Call me Hertz or blissmen
norbertbede: hi. i have created m_inventory_header_v view incuding _vt basedon other examples.
norbertbede: i cant dind but i got the follow message when start to print
norbertbede: m_inventory_header_v_id
norbertbede: any idea what is the root cause
tbayen: Does your view have an ID column?
norbertbede: yes m_inventory_id
blissmen: could u past your view code?
norbertbede: SELECT i.ad_client_id,
norbertbede: i.ad_org_id,
norbertbede: i.isactive,
norbertbede: i.created,
norbertbede: i.createdby,
norbertbede: blissmen pasted
tbayen: I am not sure but can it be that you need a column like m_inventory_header_v_id? Did you look into the other views how they do it?
tbayen: (I am sure you did... hmmmmm)
norbertbede: i got a tip m_inventory_id as m_inventory_header_v_id
norbertbede: :) not yet tested probably helps
tbayen: Yep. That is what I proposed.
mde_: hi
tbayen: hi
mde_: i have problem launching my idempiere in gui from eclipse
mde_: this is the errror
mde_: at java.awt.GraphicsEnvironment.checkHeadless(GraphicsEnvironment.java:207) at java.awt.Window.<init>(Window.java:535) at java.awt.Frame.<init>(Frame.java:420) at org.compiere.util.Splash.<init>(Splash.java:96) at org.compiere.util.Splash.getSplash(Splash.java:82) at org.compiere.util.Splash.getSplash(Splash.java:71) at org.compiere.AdempiereClient.main(AdempiereClient.java:180) at org.adempiere.client.Application.start(App
tbayen: Do you work with the Swing or the web client?
mde_: yes swing
mde_: web client is fine
CarlosRuiz: mde_, maybe you're running eclipse as root (like with sudo) within a user environment?
tbayen: What is the first line of the error (the exception before "at ....")?
mde_: i run it with out sudo
mde_: before the at "java.awt.HeadlessException"
mde_: before the at there is "java.awt.HeadlessException"
tbayen: Normally with this exception there is the text "No X11 DISPLAY variable was set, but this program performed an operation which requires it.". Is it there too?
tbayen: Are you somehow using a network, start it via ssh on another machine or doing somthing else remotely?
mde_: the errror is shown in Splash constructor when try to call the constructor of super class. in "super("iDempiere");"
mde_: @tbayen , no there isn't this text
mde_: i can acces remotly just when i used web client
tbayen: Your Java VM thinks it is running "headless". That means without a configured output display. This is not an issue of iDempiere but of the environment you use.
tbayen: That is why we ask for sudo/remote and the like.
tbayen: Did you change something in the eclipse launch configuration? If you configure the runtime environment wrong it may cause this error.
mde_: could please tel me how can i fixe it :)
mde_: no, i didnt
tbayen: With linux this often belongs to an unset DISPLAY variable. I suspect you are running windows. Then I have no clue.
mde_: i run ubuntu 12.10
tbayen: You can look into the launch configuration with "Run -> Debug configurations...". If you are a newbie chances are that you never looked into that window. :-) If not: Did you mess somewhere? Have you changed something on the "Environment" tab of the "swingclient.product" configuration?
tbayen: Hmmm.. you mostly have to visit this window to make it run. In most versions you have to repair the plugin list (Add required Plug-ins). Did you do that?
mde_: that's happened evry time i try to open swing function
mde_: runserver or generate model
tbayen: Did you visit the "Debug Configurations" window (Or "Run configurations" - it's the same)?
a42niem: mde_: maybe http://stackoverflow.com/questions/5362512/unable-to-run-java-gui-programs-with-ubuntu helps
nmicoud: hi everyone ! can someone have a look there please : https://groups.google.com/forum/?fromgroups#!topic/idempiere/CxFbQmGeofw ; i don't know if i find a bug or if i just need to go to bed ?
CarlosRuiz: nmicoud, looks similar to another recent finding from you - when you're editing in detail panel the context variables are set to the master record
CarlosRuiz: we solved it using @1|C_TaxCategory_ID@
nmicoud: yes, but it was on the process parameter panel
nmicoud: and seems that the @x|..._ID@ notation doesn't work in dynamic validation
nmicoud: or i was not able to make it work
CarlosRuiz: must work - is the same context variable passed to parameter panel
nmicoud: so, the bug is between the chair and the keyboard ?
nmicoud: will try again then
CarlosRuiz: :-) the interface
nmicoud: nope, it doesn't work : here the dynamic validation i'm using :
nmicoud: C_Tax.C_TaxCategory_ID=@1|C_TaxCategory_ID@
nmicoud: do you see something wrong ?
CarlosRuiz: you can check the context and see if there is a variable with those values
CarlosRuiz: 1|C_TaxCategory_ID
nmicoud: => 2|1|C_TaxCategory_ID == 1000000
CarlosRuiz: yep - that's correct
nmicoud: but if i change, the value of the tax category, the tax list is not updated
nmicoud: but the context is updated
nmicoud: eg: 2|1|C_TaxCategory_ID == 1000009
CarlosRuiz: interesting :-)
nmicoud: yes :)
CarlosRuiz: must be related with resolution of dependent fields
nmicoud: i think
CarlosRuiz: look at GridField line 304
CarlosRuiz: Evaluator.parseDepends(list, m_lookup.getValidation());
nmicoud: it will never do something
CarlosRuiz: so, maybe a better discovery is needed in GridTab.hasDependants and getDependantFields ?
nmicoud: why m_lookup.getValidation() return "" ? (on GridField line 304) ?
CarlosRuiz: is a method to be overridden
CarlosRuiz: must use MLookup when running
nmicoud: ah yes
nmicoud: the parseDepends in Evaluator doesn't care about the |
nmicoud: Carlos, it's getting late here and i will have to go ; should i open a jira ticket ?
CarlosRuiz: yes, I think capturing in detail must be broken because of that
nmicoud: ok, will create one then
nmicoud: thanks Carlos, bye bye
CarlosRuiz: bye
tbayen: CarlosRuiz, if you like we can come back to IDEMPIERE-2104a. I tested it and improved the patch.
tbayen: I tested it in some different scenarios and it works well.
tbayen: For me - you tend to find some issue in every patch... :-{
tbayen: :-)
tbayen: Fine CarlosRuiz thanks! :-) Now that I get a ReplyTo mail to the gatekeeper... what do you think about IDEMPIERE-2132? Is there any good reason to not do it?
CarlosRuiz: ah - precisely reading about that
CarlosRuiz: http://tools.ietf.org/html/rfc2112#section-4
CarlosRuiz: it seems safe - I'll commit it
CarlosRuiz: hmmm - wait a minute
CarlosRuiz: your patch is not valid
tbayen: As I understand it "related" is for things that really belong together. Like a html page and its images. But the PDF is an document of its own. It can live without the mail text and the mail text can live without the PDF.
CarlosRuiz: line 983 is alreadyMultipart mp = new MimeMultipart("related");
CarlosRuiz: so, maybe your patch is for an older version of the class?
tbayen: When I did it it was actual.
tbayen: Ah stop - I remember!
tbayen: Once I had the feeling that I used the "hg export" command and chenged the revisions "a b" to "b a". I realized that later but could not find the wrong patch. Then I thought that I did it not wrong.
tbayen: It can be that the patch is "the wrong way around".
CarlosRuiz: :-)
CarlosRuiz: can you please test it again - changing related to mixed (as your comment says)
tbayen: If you do it your own and break it you can use my patch to repair it again! :-)
CarlosRuiz: and great if you can test it forwarding on thunderbird, but also forwarding on a gmail session on browser
tbayen: I use "mixed" since some weeks. And very successful.
CarlosRuiz: wow - is late on your side
tbayen: My code is ok. It is just that I created the patch file in a wrong way.
CarlosRuiz: ok, committing
tbayen: Yeah, it's late. But I feel strong that I am again collecting patches here. And I wanted to be sure that some of them will get into trunk. This is a lucky day for my "private branch". It will die soon.
tbayen: While waiting for an upload I had time to do IDEMPIERE-2180. We talked about that.
CarlosRuiz: yep - I showed the same to Andrea in Italy - but I created an idempiere-server-debug.sh :-) your approach looks better
tbayen: :-)
tbayen: If you like you can commit also IDEMPIERE-2161 in your version. :-)
tbayen: I just solved it fast because I needed it. You understanding of these classes playing together is still better than mine.
tbayen: I have one more patch in my "private branch". I am not sure if it is for trunk. It belongs to IDEMPIERE-2104 and it allows to have more than one BCC recipient. WDYT?
CarlosRuiz: separated by commas?
tbayen: yes
CarlosRuiz: sounds good
tbayen: ok. I have to prepare it in a nicer way. I will make a ticket now and add the patch in the next days.
tbayen: After that I can delete my "private branch". That was my goal - not to have a private branch for longer than some weeks. :-)
tbayen: I have to review IDEMPIERE-2074 in the next week for our "iDempiere light" project. If someone has any comments (or additions) on that you are welcome.
tbayen: gtg. good night"
tbayen: gtg. good night!
CarlosRuiz: bye
tbayen: And thanks for the great progress today! :-)
CarlosRuiz: tbayen, you're welcome