The problem was resolved and she thanked us. I'm satisfied. -Adrian
--- On Thu, 5/27/10, Hans Bakker <[email protected]> wrote: > From: Hans Bakker <[email protected]> > Subject: Re: How to I overcome this error? > To: [email protected] > Date: Thursday, May 27, 2010, 6:06 PM > Wouldn't apologies from someone > presenting herself as an expert and > causing so much work for several people be nice here? > > > On Thu, 2010-05-27 at 13:20 -0400, Ruth Hoffman wrote: > > Hi Tim: > > I just did it again and you and Adrian are correct. It > is not in there... > > Thanks for your help. > > Regards, > > Ruth > > > > Tim Ruppert wrote: > > > Here are those same two methods that I got from > the download > > > -- NOTE I see no "// @Override" > > > > > > // ----- > TemporalExpressionVisitor Implementation ----- // > > > > > > public void > visit(Difference expr) { > > > > VisitorState newState = new > VisitorState(); > > > > newState.isIntersection = > this.state.isIntersection; > > > > this.stateStack.push(this.state); > > > this.state > = newState; > > > > expr.getIncluded().accept(this); > > > > newState.isExcluded = true; > > > > expr.getExcluded().accept(this); > > > this.state > = this.stateStack.pop(); > > > if > (this.state.isIntersection) { > > > > this.state.inclRecurList.addAll(newState.inclRecurList); > > > > this.state.exRecurList.addAll(newState.exRecurList); > > > } > > > } > > > > > > > �...@suppresswarnings("unchecked") > > > public void > visit(HourRange expr) { > > > NumberList > hourList = new NumberList(); > > > > hourList.addAll(expr.getHourRangeAsSet()); > > > Recur recur > = new Recur(Recur.HOURLY, 0); > > > > recur.getHourList().addAll(hourList); > > > > this.state.addRecur(recur); > > > } > > > > > > Hope this helps Ruth. > > > > > > Cheers, > > > Ruppert > > > > > > On May 27, 2010, at 10:46 AM, Adrian Crum wrote: > > > > > > > > >> It would be interesting to see if anyone else > can find those annotations in that download. I certainly > can't. > > >> > > >> -Adrian > > >> > > >> On 5/27/2010 9:27 AM, Ruth Hoffman wrote: > > >> > > >>> Hi Adrian: > > >>> They may not be in SVN, but they > certainly are in this download. I just > > >>> double checked: > > >>> > > >>> http://apache.multihomed.net/ofbiz/apache-ofbiz-09.04.zip > > >>> > > >>> Regards, > > >>> Ruth > > >>> > > >>> Adrian Crum wrote > > >>> > > >>>> I checked the 9.04 release zip file > and the repository - those > > >>>> annotations do not exist in 9.04. > They were removed last March - so > > >>>> maybe you are using an old download. > At any rate, the problem doesn't > > >>>> exist in current code. > > >>>> > > >>>> -Adrian > > >>>> > > >>>> On 5/27/2010 8:57 AM, Ruth Hoffman > wrote: > > >>>> > > > >>>>> Hi Adrian: > > >>>>> > > >>>>> I just downloaded the 9.04 > release from the download site and these > > >>>>> lines are in the > iCalRecurConverter.java source starting at line 124: > > >>>>> > > >>>>> @Override > > >>>>> public void visit(Difference > expr) { > > >>>>> VisitorState newState = new > VisitorState(); > > >>>>> newState.isIntersection = > this.state.isIntersection; > > >>>>> > this.stateStack.push(this.state); > > >>>>> this.state = newState; > > >>>>> expr.getIncluded().accept(this); > > >>>>> ... > > >>>>> > > >>>>> Thanks. > > >>>>> Regards > > >>>>> Ruth > > >>>>> > > >>>>> Adrian Crum wrote: > > >>>>> > > > >>>>>> Where are you getting that > file from? Those annotations are not in > > >>>>>> 9.04. > > >>>>>> > > >>>>>> -Adrian > > >>>>>> > > >>>>>> On 5/27/2010 8:40 AM, Ruth > Hoffman wrote: > > >>>>>> > > > >>>>>>> Hi Adrian: > > >>>>>>> Here you go...a partial > listing from recent 9.04 > > >>>>>>> iCalRecurConverter.java > > >>>>>>> (with my comments) > source: > > >>>>>>> > > >>>>>>> // ----- > TemporalExpressionVisitor Implementation ----- // > > >>>>>>> > > >>>>>>> // @Override > > >>>>>>> public void > visit(Difference expr) { > > >>>>>>> VisitorState newState = > new VisitorState(); > > >>>>>>> newState.isIntersection = > this.state.isIntersection; > > >>>>>>> > this.stateStack.push(this.state); > > >>>>>>> this.state = newState; > > >>>>>>> > expr.getIncluded().accept(this); > > >>>>>>> newState.isExcluded = > true; > > >>>>>>> > expr.getExcluded().accept(this); > > >>>>>>> this.state = > this.stateStack.pop(); > > >>>>>>> if > (this.state.isIntersection) { > > >>>>>>> > this.state.inclRecurList.addAll(newState.inclRecurList); > > >>>>>>> > this.state.exRecurList.addAll(newState.exRecurList); > > >>>>>>> } > > >>>>>>> } > > >>>>>>> > > >>>>>>> > @SuppressWarnings("unchecked") > > >>>>>>> // @Override > > >>>>>>> public void > visit(HourRange expr) { > > >>>>>>> NumberList hourList = new > NumberList(); > > >>>>>>> > hourList.addAll(expr.getHourRangeAsSet()); > > >>>>>>> Recur recur = new > Recur(Recur.HOURLY, 0); > > >>>>>>> > recur.getHourList().addAll(hourList > > >>>>>>> ... > > >>>>>>> TIA > > >>>>>>> Ruth > > >>>>>>> Adrian Crum wrote: > > >>>>>>> > > > >>>>>>>> On 5/27/2010 4:18 AM, > Ruth Hoffman wrote: > > >>>>>>>> > > > >>>>>>>>> (Ended up just > commenting out the offending lines in the code.) > > >>>>>>>>> > > > > >>>>>>>> I would be interested > in seeing what those offending lines are. > > >>>>>>>> Like I > > >>>>>>>> said in my previous > reply, there are no @Override annotations in the > > >>>>>>>> 9.04 version of that > file. > > >>>>>>>> > > >>>>>>>> -Adrian > > >>>>>>>> > > >>>>>>>> > > > > > > > > > > > > -- > Ofbiz on twitter: http://twitter.com/apache_ofbiz > Myself on twitter: http://twitter.com/hansbak > Antwebsystems.com: Quality services for competitive rates. > >
