IDempiere/FullMeeting20140423

From WikiQSS

Table of Contents | Full Meeting Minutes | Full Meeting 2014-04-23

CarlosRuiz: Good morning
tbayen: Daarestiet
hieplq: @CarlosRuiz. I still redo this tracker http://sourceforge.net/p/adempiere/zk-web-client/252/ in idempiere.
hieplq: I don't know it's error or non
hieplq: you can hepl me check it?
hieplq: i redo with close action of standart order with garden user role
red1: its an old bug?
hieplq: I don't no it's bug or no :)
CarlosRuiz: sounds like error - let me check ...
hieplq: you can check logic in line 2584 class MRole
CarlosRuiz: does the close happen?
hieplq: yes. close is show in list action and do success
aguerra: Hello every body!!!
CarlosRuiz: Hi Alejandro
CarlosRuiz: hieplq, in such case is a bug - a different one probably as the description there is different
hieplq: you see it's a bug?
CarlosRuiz: testing ...
CarlosRuiz: yep - definitely a bug
hieplq: just change 2586 to "} else if ("Y".equals(active)){" wdyt?
hieplq: Class MRole.
CarlosRuiz: yep - I was patching it like
CarlosRuiz: if ("N".equals(active)) {
CarlosRuiz: if (validOptions.contains(op)) {
CarlosRuiz: validOptions.remove(op);
CarlosRuiz: }
CarlosRuiz: } else {
CarlosRuiz: if (!validOptions.contains(op)) {
CarlosRuiz: validOptions.add(op);
CarlosRuiz: }
CarlosRuiz: }
proSENSE2: Hell-Oh
proSENSE2: so much people here today
CarlosRuiz: Hi proSENSE2 - we usually hang around here on wednesdays to check dev issues and tickets
proSENSE2: oh, i see :)
ocurieles_DCS: Hi @CarlosRuiz
CarlosRuiz: Hi Orlando
ocurieles_DCS: some News about this ?? https://idempiere.atlassian.net/browse/IDEMPIERE-1881?filter=-2
ocurieles_DCS: And This ? https://idempiere.atlassian.net/browse/IDEMPIERE-1817?filter=-2
ocurieles_DCS: :D
CarlosRuiz: nope - have you checked for a patch?
ocurieles_DCS: mm nop :(
CarlosRuiz: in these meetings I can help your developers for clues about where to patch
ocurieles_DCS: (y)
hieplq: @CarlosRuiz, you still don't check my report about security?
CarlosRuiz: not yet - checking at this moment IDEMPIERE-1696
CarlosRuiz: you have contributed a lot! I'm struggling to keep your pace ;-)
CarlosRuiz: hieplq, with latest changes the 1696 is not working for me - still throwing the error about missing class org.bouncycastle.asn1.ASN1OctetString
hieplq: huh. I will retest it and feedback late.
TurkerTunali: hi, what is the default password for System user?
hieplq: System
TurkerTunali: I want to check translation options
TurkerTunali: ok, it is case sensitive
nmicoud: Hi CarlosRuiz : any chance you can have a look at https://idempiere.atlassian.net/browse/IDEMPIERE-1899 ? (still at draft stage, to say if if it's in the good way)
CarlosRuiz: ok - let me finish 1696 and will move to 1899 pre-check
nmicoud: sure, thanks
CarlosRuiz: hieplq, do you still have the original patch on org.adempiere.base?
hieplq: yes. i will upload it, error is by class PdfEncryption in base use this lib
CarlosRuiz: yep - iText
hieplq: @CarlosRuiz. I upload old patch
nmicoud_: gtg ; will come back tonight ; bye
CarlosRuiz: hieplq, tests went fine
CarlosRuiz: sad that there are no properties to set permissions on pdf
CarlosRuiz: maybe we could set an idempiere property on the jrxml and programatically set the permissions?
CarlosRuiz: is that worthy?
hieplq: we will improve it. current i have no idea. but in preview or document is not compleate I want output a file without print permission
red1: I have to go to Vietnam again to slow hieplq down :)
hieplq: red1. i being slow down by learning zk theme :). i find too much template to see message is hard code.
red1: yes its too hard for me too
red1: i better do business logic plugins,
hieplq: it make me discouraged
hieplq: hi CarlosRuiz. below patch, sometime make my respository conflict. when you have time, please help me review it. https://idempiere.atlassian.net/browse/IDEMPIERE-1555
CarlosRuiz: hieplq, do you mean the actual code causes issues, or after the patch?
hieplq: this code is ok. but sometime other people make patch relate process, it make my code conflict when meged with core
hieplq: because if you see it's good too make common, please review it. other i just use it in my code.
CarlosRuiz: hieplq, the class DocumentLink is new, yours
hieplq: yes.
CarlosRuiz: so, better you put correct attribution of contributors and sponsors
CarlosRuiz: on the GPL header I mean
CarlosRuiz: it's mentioned Schaeffer - and Idalica
hieplq: ah. I know. will fix it