IDempiere/FullMeeting20131204

⇐ Table of Contents | Full Meeting Minutes | Full Meeting 2013-12-04

CarlosRuiz: Good Morning
nmicoud: Bonjour
nmicoud: Found and reported some bugs this week :) ; can we deal with 1587 ?
ocurieles: Good Morning Srs
CarlosRuiz: Hi Orlando
nmicoud: Hi
CarlosRuiz: sure nmicoud, please let me integrate some commits from hengsin first ...
nmicoud: sure
ocurieles_: @JanThielemann Nice Asterisk release...
JanThielemann: please test and report bugs :)
JanThielemann: i know there are some of them xD
JanThielemann: or at least i cannot imagine that there are none *g*
CarlosRuiz: yes, JanThielemann and jmpiloq - thanks for the contribution! it looks great
ocurieles_: Sure.. i have a asterisk in my office...
JanThielemann: improvement suggestions are welcome :)
ocurieles_: please check this https://bitbucket.org/dcs_bitbucket/attachmentscanner
ocurieles_: for scanning directly from idempiere as attachment
ocurieles_: work with sane
aguerra: Hi everybody!!!!
ocurieles_: Hi @aguerra
nmicoud: it's xmas ! congrats !
CarlosRuiz: Hi Alejandro
CarlosRuiz: cybermonday contribs :-)
tbayen: Hi
ocurieles_: :D
CarlosRuiz: hi Thomas
nmicoud: hi thomas
ocurieles_: Hi Thomas
aguerra: Hi Thomas
aguerra: Hi @ocurieles_
aguerra: hi @CarlosRuiz
ocurieles_: i put the scanning images on Facebook https://www.facebook.com/groups/idempiere/
ocurieles_: somebody know a good irc client for Mac ?
ocurieles_: @JanThielemann maybe you can put some pictures of Asterisk integration on Facebook :D
JanThielemann: limechat
JanThielemann: sure
tbayen: Better make a wikipage with some pictures. It will last longer than facebook.
ocurieles_: sure i promess work the next days in the wiki plugins pages :(
hengsin: I think we should make a page in wiki to list extensions available
ocurieles_: i will work on that i pro mess :D
CarlosRuiz: advance on that task was this:
tbayen: Someone wanted to create a "plugin shop" page in the wiki. But until someone does that a Category "Plugins" could be sufficient.
CarlosRuiz: http://wiki.idempiere.org/en/Category:AvailablePlugins
CarlosRuiz: and installation of the rating plugin as shown here
CarlosRuiz: http://wiki.idempiere.org/en/User:TBayen
CarlosRuiz: so, I think is just to start filling pages and assigning that category
ocurieles_: sure
CarlosRuiz: nmicoud, finished the integration
nmicoud: ok
nmicoud: so, i think there is a bug as is wrote in my 2nd comment and 'old' columns should be updated
CarlosRuiz: yep - so, to generate the migration script we need to fix first the code
CarlosRuiz: right?
nmicoud: yep
ocurieles_DCS: testing limechat
nmicoud: about the fix, what is the interest to have CHAR with length > 1 ?
CarlosRuiz: AFAIR - char columns are just problematic in lists
nmicoud: that would explain why docaction is set as Char
hengsin: updating the old columns from char to nvarchat will break many views though
ocurieles_DCS: Hi Mr @hengsin some news about https://idempiere.atlassian.net/browse/IDEMPIERE-1531 ?
ocurieles_DCS: the field New Accumulated Amt is for accumulated inventory cost or accumulated inventory value
ocurieles_DCS: ?
CarlosRuiz: nmicoud, this query is showing 55 different columns
CarlosRuiz: select column_name, data_length, count(*) from user_tab_columns
CarlosRuiz: where data_type='CHAR' and data_length>1
CarlosRuiz: group by column_name, data_length
CarlosRuiz: order by 3 desc, 1
CarlosRuiz: -> oracle query
nmicoud: yep ; that's a lot :-/
nmicoud: should be update all of them ?
CarlosRuiz: or maybe we add an exception for DocAction and DocStatus
nmicoud: only for docaction
CarlosRuiz: ah - because is a button
nmicoud: new column for docstatus are ok
nmicoud: yes
nmicoud: only 'old' docstatus need to be updated
CarlosRuiz: no, the opposite
nmicoud: yes :D
hengsin: dcs, that's to accumulate inventory cost
CarlosRuiz: nmicoud, AD_LANGUAGE - we have 17 CHAR and 93 VARCHAR2
CarlosRuiz: :-) worst -> 17 char(5), 93 varchar2(6)
nmicoud: should we laugh or cry ?
CarlosRuiz: so, I think the problem you found is not because char<>varchar - but because of the N on nvarchar
CarlosRuiz: and would be just oracle related
nmicoud: those type are not available on postgre ?
CarlosRuiz: nope - nvarchar is oracle specific - to use national character set instead of utf8
nmicoud: ok
CarlosRuiz: so - a possible fix is to change line 611
CarlosRuiz: return "NVARCHAR2(" + fieldLength + ")";
CarlosRuiz: must be
CarlosRuiz: return "VARCHAR2(" + fieldLength + ")";
nmicoud: that will not effect DocAction columns
CarlosRuiz: your problem is with DocStatus
nmicoud: yes
nmicoud: means that the problem is that new DocStatus are created with the wrong type ?
nmicoud: right ?
CarlosRuiz: actually we have 9 docstatus nvarchar2 and 68 char
nmicoud: i'm testing the patch locally
nmicoud: that would mean that the new DocStatus columns (created as NVARCHAR2) should be modified to VARCHAR2 ?
CarlosRuiz: I think that would make it compatible to use it in a view
nmicoud: let me check : C_Invoice.DocStatus => become VARCHAR2 and myNewTable.DocStatus => stay NVARCHAR2 ?
CarlosRuiz: reading the documentation about nvarchar2 vs varchar2 - seems like nvarchar2 doesn't represent really an advantage - just adding problems :-)
CarlosRuiz: your new table must be varchar2 to if you plan to join those in a view
nmicoud: so i will have to update its type because it has already been created as NVARCHAR2
nmicoud: but i could deal with that
CarlosRuiz: yep - if you sync the column after the patch - must conver it
nmicoud: it seems to work
nmicoud: i mean, patch of line 611
CarlosRuiz: TkTech service is not notifying today JIRA or commits
TkTech: CarlosRuiz: Should respond on your next commit. Thanks for the heads up.
CarlosRuiz: excellent - thanks for your quick answer
CarlosRuiz: nmicoud, reviewing IDEMPIERE-1591
nmicoud: thanks
CarlosRuiz: now I see who deleted the role for SuperUser :D
nmicoud: :D
CarlosRuiz: yesterday red1 could not login as SuperUser and we found the role deleted :-DDD
nmicoud: i'm guilty ;)
red1: Aha.... alors
CarlosRuiz: I see why you cannot save
red1: It was good i tested the night before
red1: Its my favorite to talk about the Felix and iUIMobile.. they are easy and quick to demonstrate
CarlosRuiz: the reason why you cannot save is because the ui is not aware that detail record has changed
tbayen: How can you see who deletes something?
CarlosRuiz: if you touch any of the fields (i.e. inactivate it and activate it again) then you can save it
nmicoud: i see
CarlosRuiz: tbayen, usually the change log saves the deletions - but in this case for user assignment it's not saved - is just for records with ID column
nmicoud: not necesseraly a bug ?
CarlosRuiz: yes - is a bug
CarlosRuiz: will change the summary and annotate the finding
nmicoud: but only related to detail tab with all mandatory field filled with default values
nmicoud: but a bug anyway
CarlosRuiz: exactly
CarlosRuiz: the same happens if the user set the preference for the tab to be opened as grid
CarlosRuiz: https://twitter.com/iDempiere/status/408250552795938816
ocurieles_DCS: I can make a user just into the system from the local network?
ocurieles_DCS: can i make a user only login from local network ?
CarlosRuiz: ocurieles_DCS, maybe a login validator
aguerra: You can allow connections only if the server certificate in the browser idempiere have installed?
CarlosRuiz: aguerra, yes - you can configure the web server (i.e. apache) like that - and give certificates using openssh
CarlosRuiz: sorry - openssl
aguerra: And only generate certificates that can connect to the server?
aguerra: Have some documentation??
CarlosRuiz: http://www.sslshopper.com/article-how-to-create-and-install-an-apache-self-signed-certificate.html
aguerra: thk a lot!!!
CarlosRuiz: I did that once with apache - it works fine - I guess is similar with nginx
aguerra: Ok. I'll try, because I'm going to have a server with 2 remote branches and I just want to connect who have the certificate installed.
aguerra: otherwise I will use openvpn
ocurieles_DCS:
nmicoud: @CarlosRuiz: could you integrate your patch for 1461 ?
CarlosRuiz: nmicoud, on 1461 I would like a peer review from hengsin_ first
nmicoud: ok
CarlosRuiz: it's also breaking a functionality that open the zoom window in New mode when the field is empty
nmicoud: ok
nmicoud: thus, any chance with 1575, 1571 and 1589 ?
CarlosRuiz: let me test 1571 and 1589
CarlosRuiz: nmicoud, which is the test case for 1571 - inactivate which column on which table?
nmicoud: columns acct type like Ch_Revenue_Acct,... all *_Acct that have been disabled beacuse useless
CarlosRuiz: ah - so, you mean we disabled the field, but not the column - if we inactivate the column then the import account throw errors - right?
nmicoud: no, the column is disabled
CarlosRuiz: ah - I see - your csv has a row with a default account deprecated
nmicoud: that could be also that :)
nmicoud: i still use the same after all years and i only add accounts, never remove some
nmicoud: but i think that lots of csv are not up to date, so this little modification can still be used
nmicoud: but message could be more like "this account is deprecated, you should remove it from your import file"
Mark___: how i can download the manual from the Adempiere?
CarlosRuiz: Mark___, welcome to iDempiere - which is your interest on the system? Functional or technical?
Mark___: yes
Mark___: Functional and technical
CarlosRuiz: then I think you must consider reading the books published
Mark___: where do I get the books?
CarlosRuiz: http://www.google.com/search?q=adempiere%20books
CarlosRuiz: thanks nmicoud - I committed your 1589
CarlosRuiz: I found in tests there are many other places to improve with similar approaches - it would be good to have deeper analysis on that
nmicoud: thanks Carlos !