Re: svn commit: r928451 - /ofbiz/trunk/framework/widget/templates/htmlScreenMacroLibrary.ftl

Posted by Adrian Crum-2 on
URL: http://ofbiz.116.s1.nabble.com/Re-svn-commit-r928451-ofbiz-trunk-framework-widget-templates-htmlScreenMacroLibrary-ftl-tp1694238p1694350.html

--- On Sun, 3/28/10, Adam Heath <[hidden email]> wrote:

> Adrian Crum wrote:
> > --- On Sun, 3/28/10, Adam Heath <[hidden email]>
> wrote:
> >> That's not a useful response.  "Just trust
> me" is not
> >> helpful.  If you
> >> know why something will work, and someone obvious
> >> doesn't(me), then
> >> just go ahead and explain it.  Doing a bunch
> of
> >> hand-waving just gives
> >> the hand-waver a feeling of superiority, and makes
> the
> >> receiver feel
> >> inferior.
> >
> > One of the things I try to do whenever the opportunity
> presents
> > itself, is to meet other OFBiz developers in person. I
> believe
> > doing so helps us understand each other better.
> >
> > One of the drawbacks with mailing list collaboration
> is the
> > inability to know the other collaborators personally.
> That leads
> > to messages having feelings and motives assigned to
> them that
> > don't belong there. This is one of those cases.
>
> You're on my short list.........
>
>
>
>
>
> of people I'd like to meet in person.
>
> > I don't feel superior to any other developer. In my
> mind we are
> > all peers. In this particular reply, I was viewing
> *you* as being
> > superior - you are a programming guru in my opinion.
> When you
> > seemed puzzled by (what I thought was) a very simple
> code
> > construct, I *assumed* you just hadn't looked at it
> carefully
> > enough and I was sure it would make sense if you just
> looked at
> > it again, or tried it out on your local copy. I would
> have
> > explained it in more detail to anyone else.
>
> I wasn't puzzled by it.  From a very quick glance, I
> knew that it
> would have problems with real-world parameters.  I was
> just trying to
> get you to explain to me what you were trying to do.

Understood. You pointed out a flaw in the logic, I agreed with you and reverted it. Then I took a little break and asked myself how I can prevent that from happening again. It boils down to me having worked as a one man programming team for so long. It was common for me to commit code like that because I knew how it worked and I understood what not to do with it. Of course, that approach doesn't fly in this project. I try not to allow myself to commit code like that, but it still happens anyway. Maybe after enough embarrassing reverts I'll finally learn my lesson. :-)

-Adrian