Hi Tim, De : Tim Ruppert À : [email protected] Envoyé : samedi 27 octobre 2007 18:28 Objet : Re: JAZ- [Fwd: Re: svn commit: r586582 - /ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java]
I guess the bigger question is what with all of the work needing to be done in the project - why does everyone seem to spend this much time on this issue? I'm like that, it's one of my flaws : I can't resist to help someone (here we say "défendre la veuve et l'orphelin" in a mocking way) My vote - just comment the method Sounds wise and move on. Not sure Andrew's already given an explanation of why he put the method in there - good enough. Was clear yes, KISS way Adrian's already told his side of the story. Great! Now everyone's been heard - the method is not hurting anyone - Here I'm not sure Adrian think the same and I would like to well understand why comment it if you think it's a good idea I will not do anything, I just want to be sure to understand. I had no time this week-end to llok at the code, but nothing is blocking right now. I have all my time on this issue: no worries, no stress. - move on to other MUCH higher priority issues! Wooo... not so loud please ... (I don't fell like sitting and looking the train passing ;o) My two cents. Thanks for your comments. Cheers Jacques Cheers, Tim -- Tim Ruppert HotWax Media http://www.hotwaxmedia.com o:801.649.6594 f:801.649.6595 On Oct 27, 2007, at 9:19 AM, Jacques Le Roux wrote: Then the ball is in Adrian's camp. And Adrian you should explain us your POV on this ML, and then maybe a vote ? Jacques De : "David E Jones" <[EMAIL PROTECTED]> I disagree with leaning on deprecation because deprecation means "do not use", "convert all code that uses to use this other thing", and "this is slated for removal in some future release". I don't think that would be accurate here... -David On Oct 26, 2007, at 3:03 AM, Jacques Le Roux wrote: And this were comments Adrian, Wickersheimer Jeremy and I put in this issue =================================================== Adrian Crum - 21/oct./07 04:15 PM David, Thanks for your input. I don't understand what you're saying though. If those methods aren't deprecated, then developers will continue to use them. This will lead to problems down the road with inconsistent data - users are going to encounter two different results for the same date/time criteria. Are you saying inconsistent data is something we should allow? =================================================== =================================================== Wickersheimer Jeremy - 21/oct./07 07:06 PM David, i understand your point but marking the methods deprecated doesn't remove them, so they can still be used. However they should not be used, as Adrian points out they do not use the correct locale so there output will be inconsistent. Marking the deprecated is a good way to say that the code using them should be migrated at, and it would also make the compiler throw out useful warnings. =================================================== =================================================== Jacques Le Roux - 21/oct./07 11:12 PM David, I agree with David and Jeremy. Deprecating and documenting it in code seems a good idea in this case. There are better chances to be read than in the Best practices Guides (pragmatic POV) which does not mean that this should not be documented at this higher level too. Is there something else we are missing ? =================================================== So I maybe misunderstood but if deprecating is the way here (you did not say anything about that) why put in the framework a static method which, according to Adrian and his noon+24h exmaple, is a bad practice ? Maybe Adrian is wrong about his example but he has done a lot of work around this issue hence it's doubtful. Mainly to second Adrian who is really alone trying to explain his view. Jacques ----- Message d'origine ----- De : "David E Jones" <[EMAIL PROTECTED]> À : <[email protected]> Envoyé : vendredi 26 octobre 2007 02:28 Objet : Re: JAZ- [Fwd: Re: svn commit: r586582 - /ofbiz/trunk/ framework/base/src/base/org/ofbiz/base/util/UtilDateTime.java] On Oct 25, 2007, at 5:16 PM, Adrian Crum wrote: Andrew, I understand what the method does. The point I'm trying to make is this: It is not needed and it provides a way to introduce inconsistent data into the project. I understand the method solves a problem for a particular client, but it's not a good thing to include in the project. There is a discussion on Jira about this: https://issues.apache.org/jira/browse/OFBIZ-1361 Yes, and Adrian you seem to have missed my point there, or maybe you disagree with me? The framework is NOT around to force stuff. Here is what I wrote there: =================================================== While I agree that this should be the best practice, there is a big difference in the framework between what we "allow" and what we "recommend". There is lots of stuff you _can_ do with the framework that is really not a good idea, thought some might disagree. Things that we recommend should be documented in the Best Practices Guide. Other things we don't want to make more difficult, IMO, that this is important because of the comment about disagreement above. There are pretty much always good reasons why we do things the way we recommend in the framework, but those recommendations have evolved over time and will continue to evolve as well, and not allowing things we don't recommend stifles this and limits opportunity to progress and improve. That is of course a generality, and there is a clear best practice here that should be documented and it probably won't ever change, but I'm still against forcing on a matter of principle. =================================================== If you have an issue with that, let's discuss that directly, and if necessary vote on it. That seems to be the difference of opinion, so let's resolve it... now. -David
