Cheers, Tim -- Tim Ruppert HotWax Media http://www.hotwaxmedia.com
o:801.649.6594 f:801.649.6595 On Nov 14, 2007, at 10:59 AM, Scott Gray wrote:
I'm not agiainst it, +1 Scott On 15/11/2007, Vince M. Clark <[EMAIL PROTECTED]> wrote:+1 Vince Clark Global Era The Freedom of Open Source [EMAIL PROTECTED] (303) 493-6723 ----- Original Message ----- From: "Adrian Crum" <[EMAIL PROTECTED]> To: dev@ofbiz.apache.orgSent: Wednesday, November 14, 2007 10:16:31 AM (GMT-0700) America/ DenverSubject: Re: release4.0: OFBIZ-1106 (in or out?)While technically it is not a bug fix, I believe it should go in anyway - since the release is intended to be widely deployed, and the problem your patch addresses might be a deal breaker forthose who are considering deploying the release. +1 for including it. -Adrian Dan Shields wrote:Thanks Jacques for helping get my patch for OFBIZ-1106 into OFBiz.Hello Devs, recently I participated with other developers to devise afix for OFBIZ-1106. The patch I submitted is now in HEAD but UNsurprisingly it has been held back from release4.0 because the acceptance criteria, I am told, accepts only bug fixes. Some would agree that release4.0 was unusable for POS for the factthat it echos the manager's and the user's password to the screen forall staff and customers to see. I don't know if any other developerhas tried to train non-computer people to use the POS application, butI have seen the genuine surprise on their faces when they saw their own password appear on the screen as they typed. It should be self-evident that this is undesirable behavior. My patch merelyreplaces the characters on the screen with asterisks; it does so in amanner that respects existing APIs employed by the OFBiz POS application, it is well-tested, cleanly applies to HEAD and release4.0, and has been tested by other ofbiz developers as well.It seems that there is some uncertainty over whether this is in fact abug fix or not. I am merely asking for additional support in deciding:"For the purposes of release4.0, is my patch for OFBIZ-1106 a bug fix?"
smime.p7s
Description: S/MIME cryptographic signature