yup, I'll fix that tomorrow too. I will also switch to the commit-then-review approach from now on.
regards, Harry 2009/5/10 <[email protected]> > Author: jalkanen > Date: Sun May 10 18:14:07 2009 > New Revision: 773375 > > URL: http://svn.apache.org/viewvc?rev=773375&view=rev > Log: > Added a FIXME > > Modified: > > > incubator/jspwiki/trunk/src/java/org/apache/wiki/action/AbstractPageActionBean.java > > Modified: > incubator/jspwiki/trunk/src/java/org/apache/wiki/action/AbstractPageActionBean.java > URL: > http://svn.apache.org/viewvc/incubator/jspwiki/trunk/src/java/org/apache/wiki/action/AbstractPageActionBean.java?rev=773375&r1=773374&r2=773375&view=diff > > ============================================================================== > --- > incubator/jspwiki/trunk/src/java/org/apache/wiki/action/AbstractPageActionBean.java > (original) > +++ > incubator/jspwiki/trunk/src/java/org/apache/wiki/action/AbstractPageActionBean.java > Sun May 10 18:14:07 2009 > @@ -74,6 +74,7 @@ > * > * @param pageName the pageName to be removed from the breadcrumb > */ > + // FIXME: Is this in the right place? Shouldn't this be a static > method in BreadcrumbsTag? > void deleteFromBreadCrumb( String pageName ) > { > HttpSession session = getContext().getRequest().getSession( false > ); > > >
