I agree with you, however this was code already present in
wicketstuff-annotation and I just leveraged it.  I can refactor it(remove
it and rework the mount) or leave it be. Thoughts?


On Wed, Mar 12, 2014 at 12:24 AM, Alexander Morozov <
notifications@github.com> wrote:

> In
> jdk-1.6-parent/wicket-mount-parent/wicket-mount-core/src/main/java/org/wicketstuff/wicket/mount/core/AnnotatedMountList.java:
>
> > + * limitations under the License.
> > + */
> > +package org.wicketstuff.wicket.mount.core;
> > +
> > +import java.util.ArrayList;
> > +
> > +import org.apache.wicket.protocol.http.WebApplication;
> > +import org.apache.wicket.request.IRequestMapper;
> > +
> > +/**
> > + * A list of {@link IRequestMapper}.
> > + *
> > + * @author Doug Donohoe
> > + * @author Ronald Tetsuo Miura
> > + */
> > +public class AnnotatedMountList extends ArrayList<IRequestMapper>
>
> In general, it is a bad idea to extend standard collections with business
> logic. Use ArrayList as implementation details instead.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/wicketstuff/core/pull/291/files#r10505198>
> .
>