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
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>
