This is an automated email from the ASF dual-hosted git repository.

juanpablo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jspwiki.git


The following commit(s) were added to refs/heads/master by this push:
     new 1f1a47440 Missing @override.
1f1a47440 is described below

commit 1f1a47440f9a3080ea1cebccbb257ef858a5f0c9
Author: Arturo Bernal <arturobern...@gmail.com>
AuthorDate: Fri Jul 15 23:06:35 2022 +0200

    Missing @override.
---
 .../apache/wiki/api/filters/BasicPageFilter.java   | 15 ++++++++-----
 .../engine/EngineLifecycleExtensionTestImpl.java   |  9 +++++---
 .../org/apache/wiki/event/WikiPageRenameEvent.java |  2 ++
 .../main/java/org/apache/wiki/LinkCollector.java   |  3 ++-
 .../src/main/java/org/apache/wiki/WikiContext.java |  2 ++
 .../src/main/java/org/apache/wiki/WikiPage.java    |  4 +++-
 .../org/apache/wiki/attachment/Attachment.java     |  9 ++++++--
 .../java/org/apache/wiki/auth/GroupPrincipal.java  |  1 +
 .../org/apache/wiki/auth/acl/AclEntryImpl.java     | 14 ++++++++----
 .../apache/wiki/auth/acl/UnresolvedPrincipal.java  |  1 +
 .../java/org/apache/wiki/auth/authorize/Role.java  |  1 +
 .../wiki/auth/login/AbstractLoginModule.java       |  7 +++++-
 .../wiki/auth/login/AnonymousLoginModule.java      |  1 +
 .../auth/login/CookieAssertionLoginModule.java     |  1 +
 .../apache/wiki/auth/login/PrincipalWrapper.java   |  1 +
 .../wiki/auth/login/UserDatabaseLoginModule.java   |  1 +
 .../wiki/auth/login/WebContainerLoginModule.java   |  1 +
 .../wiki/auth/permissions/AllPermission.java       |  5 ++++-
 .../auth/permissions/AllPermissionCollection.java  |  5 ++++-
 .../wiki/auth/permissions/GroupPermission.java     |  2 ++
 .../wiki/auth/permissions/PagePermission.java      |  4 +++-
 .../wiki/auth/permissions/WikiPermission.java      |  3 +++
 .../org/apache/wiki/pages/PageTimeComparator.java  |  3 ++-
 .../org/apache/wiki/render/CleanTextRenderer.java  |  1 +
 .../org/apache/wiki/render/CreoleRenderer.java     |  1 +
 .../wiki/render/CustomXMLOutputProcessor.java      |  3 ++-
 .../apache/wiki/render/WysiwygEditingRenderer.java |  1 +
 .../java/org/apache/wiki/render/XHTMLRenderer.java |  1 +
 .../java/org/apache/wiki/search/SearchResult.java  |  1 +
 .../apache/wiki/search/SearchResultComparator.java |  3 ++-
 .../main/java/org/apache/wiki/tags/CookieTag.java  |  2 ++
 .../org/apache/wiki/tags/EditorIteratorInfo.java   |  1 +
 .../org/apache/wiki/tags/HistoryIteratorInfo.java  |  3 ++-
 .../org/apache/wiki/tags/IfNoSearchResultsTag.java |  1 +
 .../org/apache/wiki/tags/IncludeResourcesTag.java  |  2 ++
 .../java/org/apache/wiki/tags/LinkToParentTag.java |  1 +
 .../java/org/apache/wiki/tags/NextVersionTag.java  |  1 +
 .../java/org/apache/wiki/tags/PageDateTag.java     |  2 ++
 .../java/org/apache/wiki/tags/PageExistsTag.java   |  1 +
 .../java/org/apache/wiki/tags/PageTypeTag.java     |  2 ++
 .../java/org/apache/wiki/tags/PageVersionTag.java  |  1 +
 .../apache/wiki/tags/SearchResultIteratorInfo.java |  3 ++-
 .../org/apache/wiki/tags/SearchResultsSizeTag.java |  1 +
 .../org/apache/wiki/tags/SearchResultsTag.java     |  1 +
 .../org/apache/wiki/tags/SetPaginationTag.java     |  2 ++
 .../src/main/java/org/apache/wiki/tags/TabTag.java |  3 +++
 .../java/org/apache/wiki/tags/TemplateDirTag.java  |  1 +
 .../java/org/apache/wiki/tags/WikiBodyTag.java     |  5 ++++-
 .../java/org/apache/wiki/tags/WikiLinkTag.java     |  4 +++-
 .../java/org/apache/wiki/tags/WikiTagBase.java     |  2 ++
 .../java/org/apache/wiki/ui/AbstractCommand.java   |  9 +++++++-
 .../main/java/org/apache/wiki/ui/GroupCommand.java |  5 ++++-
 .../main/java/org/apache/wiki/ui/PageCommand.java  |  5 ++++-
 .../java/org/apache/wiki/ui/RedirectCommand.java   |  5 ++++-
 .../wiki/ui/progress/DefaultProgressManager.java   |  9 ++++++--
 .../org/apache/wiki/workflow/AbstractStep.java     | 25 +++++++++++++++++-----
 .../java/org/apache/wiki/workflow/Decision.java    |  4 +++-
 .../apache/wiki/workflow/SimpleNotification.java   |  1 +
 .../org/apache/wiki/workflow/SystemPrincipal.java  |  1 +
 .../main/java/org/apache/wiki/workflow/Task.java   |  1 +
 jspwiki-main/src/test/java/SamplePlugin2.java      |  3 ++-
 .../org/apache/wiki/auth/SecurityEventTrap.java    |  1 +
 .../java/org/apache/wiki/plugin/SamplePlugin3.java |  3 ++-
 .../java/org/apache/wiki/web/TestContainer.java    |  1 +
 .../java/org/apache/wiki/workflow/TaskTest.java    |  6 ++++--
 .../main/java/org/apache/wiki/util/MailUtil.java   |  1 +
 .../java/org/apache/wiki/util/PriorityList.java    |  3 +++
 .../wiki/util/comparators/CollatorComparator.java  |  1 +
 .../wiki/util/comparators/HumanComparator.java     |  1 +
 .../util/comparators/JavaNaturalComparator.java    |  1 +
 70 files changed, 188 insertions(+), 43 deletions(-)

diff --git 
a/jspwiki-210-adapters/src/main/java/org/apache/wiki/api/filters/BasicPageFilter.java
 
b/jspwiki-210-adapters/src/main/java/org/apache/wiki/api/filters/BasicPageFilter.java
index dc44e28d2..10b25af73 100644
--- 
a/jspwiki-210-adapters/src/main/java/org/apache/wiki/api/filters/BasicPageFilter.java
+++ 
b/jspwiki-210-adapters/src/main/java/org/apache/wiki/api/filters/BasicPageFilter.java
@@ -52,7 +52,8 @@ public class BasicPageFilter extends BasePageFilter {
         return content;
     }
 
-    public String preTranslate( final Context wikiContext, final String 
content ) throws FilterException {
+    @Override
+    public String preTranslate(final Context wikiContext, final String content 
) throws FilterException {
         final Method m = methodOfNonPublicAPI( this, "preTranslate", 
"org.apache.wiki.WikiContext", "java.lang.String" );
         return executePageFilterPhase( () -> content, m, this, wikiContext, 
content );
         // return content;
@@ -62,7 +63,8 @@ public class BasicPageFilter extends BasePageFilter {
         return htmlContent;
     }
 
-    public String postTranslate( final Context wikiContext, final String 
htmlContent ) throws FilterException {
+    @Override
+    public String postTranslate(final Context wikiContext, final String 
htmlContent ) throws FilterException {
         final Method m = methodOfNonPublicAPI( this, "postTranslate", 
"org.apache.wiki.WikiContext", "java.lang.String" );
         return executePageFilterPhase( () -> htmlContent, m, this, 
wikiContext, htmlContent );
         // return htmlContent;
@@ -72,7 +74,8 @@ public class BasicPageFilter extends BasePageFilter {
         return content;
     }
 
-    public String preSave( final Context wikiContext, final String content ) 
throws FilterException {
+    @Override
+    public String preSave(final Context wikiContext, final String content ) 
throws FilterException {
         final Method m = methodOfNonPublicAPI( this, "preSave", 
"org.apache.wiki.WikiContext", "java.lang.String" );
         return executePageFilterPhase( () -> content, m, this, wikiContext, 
content );
         // return content;
@@ -81,7 +84,8 @@ public class BasicPageFilter extends BasePageFilter {
     public void postSave( final WikiContext wikiContext, final String content 
) throws FilterException {
     }
 
-    public void postSave( final Context wikiContext, final String content ) 
throws FilterException {
+    @Override
+    public void postSave(final Context wikiContext, final String content ) 
throws FilterException {
         final Method m = methodOfNonPublicAPI( this, "postSave", 
"org.apache.wiki.WikiContext", "java.lang.String" );
         executePageFilterPhase( () -> null, m, this, wikiContext, content );
         // empty method
@@ -90,7 +94,8 @@ public class BasicPageFilter extends BasePageFilter {
     public void destroy( final WikiEngine engine ) {
     }
 
-    public void destroy( final Engine engine ) {
+    @Override
+    public void destroy(final Engine engine ) {
         final Method m = methodOfNonPublicAPI( this, "destroy", 
"org.apache.wiki.WikiEngine" );
         executePageFilterPhase( () -> null, m, this, engine );
         // empty method
diff --git 
a/jspwiki-api/src/test/java/org/apache/wiki/api/engine/EngineLifecycleExtensionTestImpl.java
 
b/jspwiki-api/src/test/java/org/apache/wiki/api/engine/EngineLifecycleExtensionTestImpl.java
index 5dbfddc03..193e09d11 100644
--- 
a/jspwiki-api/src/test/java/org/apache/wiki/api/engine/EngineLifecycleExtensionTestImpl.java
+++ 
b/jspwiki-api/src/test/java/org/apache/wiki/api/engine/EngineLifecycleExtensionTestImpl.java
@@ -25,17 +25,20 @@ import java.util.Properties;
 public class EngineLifecycleExtensionTestImpl implements 
EngineLifecycleExtension {
 
     /** {@inheritDoc */
-    public void onInit( final Properties properties ) {
+    @Override
+    public void onInit(final Properties properties ) {
         properties.put( "test", "onInit" );
     }
 
     /** {@inheritDoc */
-    public void onStart( final Engine e, final Properties properties ) {
+    @Override
+    public void onStart(final Engine e, final Properties properties ) {
         properties.put( "test", "onStart" );
     }
 
     /** {@inheritDoc */
-    public void onShutdown( final Engine e, final Properties properties ) {
+    @Override
+    public void onShutdown(final Engine e, final Properties properties ) {
         properties.put( "test", "onShutdown" );
     }
     
diff --git 
a/jspwiki-event/src/main/java/org/apache/wiki/event/WikiPageRenameEvent.java 
b/jspwiki-event/src/main/java/org/apache/wiki/event/WikiPageRenameEvent.java
index e1a54be8c..8d6696ea6 100644
--- a/jspwiki-event/src/main/java/org/apache/wiki/event/WikiPageRenameEvent.java
+++ b/jspwiki-event/src/main/java/org/apache/wiki/event/WikiPageRenameEvent.java
@@ -81,6 +81,7 @@ public class WikiPageRenameEvent extends WikiPageEvent {
      *
      * @return a String representation of the type
      */
+    @Override
     public String eventName() {
         return "PAGE_RENAMED";
     //  switch ( getType() )
@@ -94,6 +95,7 @@ public class WikiPageRenameEvent extends WikiPageEvent {
     /** Returns a human-readable description of the event type.
      * @return a String description of the type
      */
+    @Override
     public String getTypeDescription() {
         return "page renamed event";
     //  switch ( getType() )
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/LinkCollector.java 
b/jspwiki-main/src/main/java/org/apache/wiki/LinkCollector.java
index dd5fc5488..809b8c99f 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/LinkCollector.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/LinkCollector.java
@@ -42,7 +42,8 @@ public class LinkCollector implements StringTransmutator {
     /**
      * {@inheritDoc}
      */
-    public String mutate( final Context context, final String in ) {
+    @Override
+    public String mutate(final Context context, final String in ) {
         m_items.add( in );
         return in;
     }
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/WikiContext.java 
b/jspwiki-main/src/main/java/org/apache/wiki/WikiContext.java
index 0a4a8a1d4..640b9ae90 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/WikiContext.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/WikiContext.java
@@ -592,6 +592,7 @@ public class WikiContext implements Context, Command {
      *
      * @return the command
      */
+    @Override
     public Command getCommand() {
         return m_command;
     }
@@ -631,6 +632,7 @@ public class WikiContext implements Context, Command {
      *  @since  2.8.0
      *  @return A deep clone of the WikiContext.
      */
+    @Override
     @SuppressWarnings("unchecked")
     public WikiContext deepClone() {
         try {
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/WikiPage.java 
b/jspwiki-main/src/main/java/org/apache/wiki/WikiPage.java
index 770938d8b..44f308795 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/WikiPage.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/WikiPage.java
@@ -202,6 +202,7 @@ public class WikiPage implements Page {
      *
      * @return The access control list.  May return null, if there is no acl.
      */
+    @Override
     public Acl getAcl() {
         return m_accessList;
     }
@@ -227,7 +228,8 @@ public class WikiPage implements Page {
      *
      * @param acl The Acl to set
      */
-    public void setAcl( final org.apache.wiki.api.core.Acl acl ) {
+    @Override
+    public void setAcl(final org.apache.wiki.api.core.Acl acl ) {
         m_accessList = ( Acl )acl;
     }
 
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/attachment/Attachment.java 
b/jspwiki-main/src/main/java/org/apache/wiki/attachment/Attachment.java
index 05db02554..4493a80f0 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/attachment/Attachment.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/attachment/Attachment.java
@@ -73,6 +73,7 @@ public class Attachment extends WikiPage implements 
org.apache.wiki.api.core.Att
      *
      * @return A String with the file name.
      */
+    @Override
     public String getFileName() {
         return m_fileName;
     }
@@ -82,7 +83,8 @@ public class Attachment extends WikiPage implements 
org.apache.wiki.api.core.Att
      *
      * @param name The name of the attachment.  Must be a legal file name 
without  the path.
      */
-    public void setFileName( final String name ) {
+    @Override
+    public void setFileName(final String name ) {
         m_fileName = name;
     }
 
@@ -91,6 +93,7 @@ public class Attachment extends WikiPage implements 
org.apache.wiki.api.core.Att
      *
      * @return String depicting the parent of the attachment.
      */
+    @Override
     public String getParentName() {
         return m_parentName;
     }
@@ -101,6 +104,7 @@ public class Attachment extends WikiPage implements 
org.apache.wiki.api.core.Att
      * @return False, if the attachment should not be cached by the user agent.
      * @since 2.5.34
      */
+    @Override
     public boolean isCacheable() {
         return m_cacheable;
     }
@@ -112,7 +116,8 @@ public class Attachment extends WikiPage implements 
org.apache.wiki.api.core.Att
      * @param value True or false, depending on whether you want this 
attachment to be cacheable or not.
      * @since 2.5.34
      */
-    public void setCacheable( final boolean value ) {
+    @Override
+    public void setCacheable(final boolean value ) {
         m_cacheable = value;
     }
 
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/GroupPrincipal.java 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/GroupPrincipal.java
index 1bad8b00b..718dff412 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/auth/GroupPrincipal.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/auth/GroupPrincipal.java
@@ -57,6 +57,7 @@ public final class GroupPrincipal implements Principal, 
Serializable
      * @return the name
      * @see java.security.Principal#getName()
      */
+    @Override
     public String getName()
     {
         return m_name;
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/acl/AclEntryImpl.java 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/acl/AclEntryImpl.java
index 3e5c056b8..87cdac7b1 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/auth/acl/AclEntryImpl.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/auth/acl/AclEntryImpl.java
@@ -52,7 +52,8 @@ public class AclEntryImpl implements AclEntry, Serializable {
      * @return <code>true</code> if the permission was added, 
<code>false</code> if the permission was
      * already part of this entry's permission set, and <code>false</code> if 
the permission is not of type PagePermission
      */
-    public synchronized boolean addPermission( final Permission permission ) {
+    @Override
+    public synchronized boolean addPermission(final Permission permission ) {
         if( permission instanceof PagePermission && findPermission( permission 
) == null ) {
             m_permissions.add( permission );
             return true;
@@ -67,7 +68,8 @@ public class AclEntryImpl implements AclEntry, Serializable {
      * @param permission the permission to be checked for.
      * @return true if the permission is part of the permission set in this 
entry, false otherwise.
      */
-    public boolean checkPermission( final Permission permission ) {
+    @Override
+    public boolean checkPermission(final Permission permission ) {
         return findPermission( permission ) != null;
     }
 
@@ -77,6 +79,7 @@ public class AclEntryImpl implements AclEntry, Serializable {
      *
      * @return the principal associated with this entry.
      */
+    @Override
     public synchronized Principal getPrincipal() {
         return m_principal;
     }
@@ -86,6 +89,7 @@ public class AclEntryImpl implements AclEntry, Serializable {
      *
      * @return an enumeration of the permissions
      */
+    @Override
     public Enumeration< Permission > permissions() {
         return m_permissions.elements();
     }
@@ -96,7 +100,8 @@ public class AclEntryImpl implements AclEntry, Serializable {
      * @param permission the permission to be removed from this entry.
      * @return true if the permission is removed, false if the permission was 
not part of this entry's permission set.
      */
-    public synchronized boolean removePermission( final Permission permission 
) {
+    @Override
+    public synchronized boolean removePermission(final Permission permission ) 
{
         final Permission p = findPermission( permission );
         if( p != null ) {
             m_permissions.remove( p );
@@ -114,7 +119,8 @@ public class AclEntryImpl implements AclEntry, Serializable 
{
      * @return true if the principal is set, false if there was already a
      * principal set for this entry
      */
-    public synchronized boolean setPrincipal( final Principal user ) {
+    @Override
+    public synchronized boolean setPrincipal(final Principal user ) {
         if( m_principal != null || user == null ) {
             return false;
         }
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/acl/UnresolvedPrincipal.java 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/acl/UnresolvedPrincipal.java
index 97c87885e..65a7967e6 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/acl/UnresolvedPrincipal.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/acl/UnresolvedPrincipal.java
@@ -52,6 +52,7 @@ public final class UnresolvedPrincipal implements Principal, 
Serializable
      * @return the name
      * @see java.security.Principal#getName()
      */
+    @Override
     public String getName()
     {
         return m_name;
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/authorize/Role.java 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/authorize/Role.java
index 1aacfb6a6..7ed2478e5 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/auth/authorize/Role.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/auth/authorize/Role.java
@@ -118,6 +118,7 @@ public final class Role implements Principal, Serializable
      *  Returns the name of the Principal.
      * @return the name of the Role
      */
+    @Override
     public String getName()
     {
         return m_name;
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/AbstractLoginModule.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/AbstractLoginModule.java
index 731b7f40e..641d1e614 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/AbstractLoginModule.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/AbstractLoginModule.java
@@ -70,6 +70,7 @@ public abstract class AbstractLoginModule implements 
LoginModule {
      * @see javax.security.auth.spi.LoginModule#abort()
      * @return True, always.
      */
+    @Override
     public final boolean abort()
     {
         removePrincipals( m_principals );
@@ -96,6 +97,7 @@ public abstract class AbstractLoginModule implements 
LoginModule {
      *         failed
      * @see javax.security.auth.spi.LoginModule#commit()
      */
+    @Override
     public final boolean commit() {
         if ( succeeded() ) {
             for ( final Principal principal : m_principals ) {
@@ -126,7 +128,8 @@ public abstract class AbstractLoginModule implements 
LoginModule {
      * @param sharedState {@inheritDoc}
      * @param options {@inheritDoc}
      */
-    public final void initialize( final Subject subject, final CallbackHandler 
callbackHandler, final Map<String,?> sharedState, final Map<String,?> options ) 
{
+    @Override
+    public final void initialize(final Subject subject, final CallbackHandler 
callbackHandler, final Map<String,?> sharedState, final Map<String,?> options ) 
{
         m_principals = new HashSet<>();
         m_subject = subject;
         m_handler = callbackHandler;
@@ -149,6 +152,7 @@ public abstract class AbstractLoginModule implements 
LoginModule {
      * @throws LoginException if the authentication fails
      * @see javax.security.auth.spi.LoginModule#login()
      */
+    @Override
     public abstract boolean login() throws LoginException;
 
     /**
@@ -158,6 +162,7 @@ public abstract class AbstractLoginModule implements 
LoginModule {
      *         <code>false</code> if this LoginModule should be ignored
      * @see javax.security.auth.spi.LoginModule#logout()
      */
+    @Override
     public final boolean logout() {
         removePrincipals( m_principals );
 
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/AnonymousLoginModule.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/AnonymousLoginModule.java
index 3051d0837..6da91a51f 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/AnonymousLoginModule.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/AnonymousLoginModule.java
@@ -71,6 +71,7 @@ public class AnonymousLoginModule extends AbstractLoginModule
      * @see javax.security.auth.spi.LoginModule#login()
      * @throws LoginException if unable to handle callback
      */
+    @Override
     public boolean login() throws LoginException
     {
         // Let's go and make a Principal based on the IP address
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/CookieAssertionLoginModule.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/CookieAssertionLoginModule.java
index bfdb7574f..ac626b1e7 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/CookieAssertionLoginModule.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/CookieAssertionLoginModule.java
@@ -73,6 +73,7 @@ public class CookieAssertionLoginModule extends 
AbstractLoginModule {
      * method throws a <code>FailedLoginException</code>.
      * @see javax.security.auth.spi.LoginModule#login()
      */
+    @Override
     public boolean login() throws LoginException {
         // Otherwise, let's go and look for the cookie!
         final HttpRequestCallback hcb = new HttpRequestCallback();
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/PrincipalWrapper.java 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/PrincipalWrapper.java
index 99f4ff67f..974a5a864 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/PrincipalWrapper.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/PrincipalWrapper.java
@@ -53,6 +53,7 @@ public final class PrincipalWrapper implements Principal
      * 
      * @return The name of the wrapped principal.
      */
+    @Override
     public String getName()
     {
         return m_principal.getName();
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/UserDatabaseLoginModule.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/UserDatabaseLoginModule.java
index 9a7a58548..97f1a8dce 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/UserDatabaseLoginModule.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/UserDatabaseLoginModule.java
@@ -62,6 +62,7 @@ public class UserDatabaseLoginModule extends 
AbstractLoginModule {
      * 
      * {@inheritDoc}
      */
+    @Override
     public boolean login() throws LoginException
     {
         final UserDatabaseCallback ucb = new UserDatabaseCallback();
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/WebContainerLoginModule.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/WebContainerLoginModule.java
index 3aef9ea4e..68f3c8dd1 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/login/WebContainerLoginModule.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/login/WebContainerLoginModule.java
@@ -69,6 +69,7 @@ public class WebContainerLoginModule extends 
AbstractLoginModule {
      * @return {@inheritDoc}
      * @throws LoginException {@inheritDoc}
      */
+    @Override
     public boolean login() throws LoginException
     {
         final HttpRequestCallback rcb = new HttpRequestCallback();
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/AllPermission.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/AllPermission.java
index fe502da37..8a12a41c8 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/AllPermission.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/AllPermission.java
@@ -73,6 +73,7 @@ public final class AllPermission extends Permission 
implements Serializable {
      * @return Always null.
      * @see java.security.Permission#getActions()
      */
+    @Override
     public String getActions() {
         return null;
     }
@@ -109,7 +110,8 @@ public final class AllPermission extends Permission 
implements Serializable {
      * <code>false</code> otherwise
      * @see java.security.Permission#implies(java.security.Permission)
      */
-    public boolean implies( final Permission permission ) {
+    @Override
+    public boolean implies(final Permission permission ) {
         // Permission must be a JSPWiki permission, PagePermission or 
AllPermission
         if( !PermissionChecks.isJSPWikiPermission( permission ) ) {
             return false;
@@ -137,6 +139,7 @@ public final class AllPermission extends Permission 
implements Serializable {
      * @return {@inheritDoc}
      * @see java.security.Permission#newPermissionCollection()
      */
+    @Override
     public PermissionCollection newPermissionCollection() {
         return new AllPermissionCollection();
     }
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/AllPermissionCollection.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/AllPermissionCollection.java
index 6d6566ddd..152ecea70 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/AllPermissionCollection.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/AllPermissionCollection.java
@@ -96,7 +96,8 @@ public class AllPermissionCollection extends 
PermissionCollection
      * 
      * @return {@inheritDoc}
      */
-    public boolean implies( final Permission permission )
+    @Override
+    public boolean implies(final Permission permission )
     {
         // If nothing in the collection yet, fail fast
         if ( !m_notEmpty )
@@ -126,6 +127,7 @@ public class AllPermissionCollection extends 
PermissionCollection
     /**
      * {@inheritDoc}
      */
+    @Override
     public boolean isReadOnly()
     {
         return m_readOnly;
@@ -134,6 +136,7 @@ public class AllPermissionCollection extends 
PermissionCollection
     /**
      * @see java.security.PermissionCollection#setReadOnly()
      */
+    @Override
     public void setReadOnly()
     {
         m_readOnly = true;
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/GroupPermission.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/GroupPermission.java
index 7731e6c18..cef601ab1 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/GroupPermission.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/GroupPermission.java
@@ -208,6 +208,7 @@ public final class GroupPermission extends Permission 
implements Serializable
      * @return the actions
      * @see java.security.Permission#getActions()
      */
+    @Override
     public String getActions()
     {
         return m_actionString;
@@ -269,6 +270,7 @@ public final class GroupPermission extends Permission 
implements Serializable
      * supplied Permission; <code>false</code> otherwise
      * @see java.security.Permission#implies(java.security.Permission)
      */
+    @Override
     public boolean implies(final Permission permission )
     {
         // Permission must be a GroupPermission
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/PagePermission.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/PagePermission.java
index 8e6b5786a..c3524d224 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/PagePermission.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/PagePermission.java
@@ -235,6 +235,7 @@ public final class PagePermission extends Permission 
implements Serializable
      *
      * @return {@inheritDoc}
      */
+    @Override
     public String getActions()
     {
         return m_actionString;
@@ -294,7 +295,8 @@ public final class PagePermission extends Permission 
implements Serializable
      * @param permission {@inheritDoc}
      * @return {@inheritDoc}
      */
-    public boolean implies( final Permission permission )
+    @Override
+    public boolean implies(final Permission permission )
     {
         // Permission must be a PagePermission
         if ( !( permission instanceof PagePermission ) )
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/WikiPermission.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/WikiPermission.java
index af475e82c..c949c5536 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/WikiPermission.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/auth/permissions/WikiPermission.java
@@ -137,6 +137,7 @@ public final class WikiPermission extends Permission 
implements Serializable
      * @return the actions
      * @see java.security.Permission#getActions()
      */
+    @Override
     public String getActions()
     {
         return m_actionString;
@@ -172,6 +173,7 @@ public final class WikiPermission extends Permission 
implements Serializable
      * <code>false</code> otherwise
      * @see java.security.Permission#implies(java.security.Permission)
      */
+    @Override
     public boolean implies(final Permission permission )
     {
         // Permission must be a WikiPermission
@@ -195,6 +197,7 @@ public final class WikiPermission extends Permission 
implements Serializable
      * Returns a new {@link AllPermissionCollection}.
      * @return {@inheritDoc}
      */
+    @Override
     public PermissionCollection newPermissionCollection()
     {
         return new AllPermissionCollection();
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/pages/PageTimeComparator.java 
b/jspwiki-main/src/main/java/org/apache/wiki/pages/PageTimeComparator.java
index 7def64ad9..01803d3a9 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/pages/PageTimeComparator.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/pages/PageTimeComparator.java
@@ -39,7 +39,8 @@ public class PageTimeComparator implements Comparator< Page 
>, Serializable {
     /**
      *  {@inheritDoc}
      */
-    public int compare( final Page p1, final Page p2 ) {
+    @Override
+    public int compare(final Page p1, final Page p2 ) {
         if( p1 == null || p2 == null ) {
             LOG.error( "W1 or W2 is NULL in PageTimeComparator!");
             return 0; // FIXME: Is this correct?
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/render/CleanTextRenderer.java 
b/jspwiki-main/src/main/java/org/apache/wiki/render/CleanTextRenderer.java
index e3e967383..e720c396e 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/render/CleanTextRenderer.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/render/CleanTextRenderer.java
@@ -43,6 +43,7 @@ public class CleanTextRenderer extends WikiRenderer {
     /**
      *  {@inheritDoc}
      */
+    @Override
     public String getString() {
        return m_document != null ? XmlUtil.extractTextFromDocument( 
m_document.getDocument() ) : "";
     }
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/render/CreoleRenderer.java 
b/jspwiki-main/src/main/java/org/apache/wiki/render/CreoleRenderer.java
index 3c40cdbb6..263295204 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/render/CreoleRenderer.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/render/CreoleRenderer.java
@@ -156,6 +156,7 @@ public class CreoleRenderer extends WikiRenderer {
     /**
      *  {@inheritDoc}
      */
+    @Override
     public String getString() {
        final StringBuilder sb = new StringBuilder(1000);
         final Element ce = m_document.getRootElement();
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/render/CustomXMLOutputProcessor.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/render/CustomXMLOutputProcessor.java
index de5cb56a4..9ec941236 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/render/CustomXMLOutputProcessor.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/render/CustomXMLOutputProcessor.java
@@ -33,7 +33,8 @@ import java.io.Writer;
  */
 public class CustomXMLOutputProcessor extends AbstractXMLOutputProcessor {
 
-    protected void attributeEscapedEntitiesFilter( final Writer out, final 
FormatStack fstack, final String value ) throws IOException {
+    @Override
+    protected void attributeEscapedEntitiesFilter(final Writer out, final 
FormatStack fstack, final String value ) throws IOException {
         write( out, Format.escapeAttribute( fstack.getEscapeStrategy(), value 
) );
     }
 
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/render/WysiwygEditingRenderer.java 
b/jspwiki-main/src/main/java/org/apache/wiki/render/WysiwygEditingRenderer.java
index 4fc389451..995011d90 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/render/WysiwygEditingRenderer.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/render/WysiwygEditingRenderer.java
@@ -132,6 +132,7 @@ public class WysiwygEditingRenderer extends WikiRenderer {
     /**
      *  {@inheritDoc}
      */
+    @Override
     public String getString() throws IOException {
         final Element rootElement = m_document.getRootElement();
         processChildren( rootElement );
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/render/XHTMLRenderer.java 
b/jspwiki-main/src/main/java/org/apache/wiki/render/XHTMLRenderer.java
index 93f0afa91..cf81ce8dd 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/render/XHTMLRenderer.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/render/XHTMLRenderer.java
@@ -50,6 +50,7 @@ public class XHTMLRenderer extends WikiRenderer {
     /**
      *  {@inheritDoc}
      */
+    @Override
     public String getString() throws IOException {
         m_document.setContext( m_context );
 
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/search/SearchResult.java 
b/jspwiki-main/src/main/java/org/apache/wiki/search/SearchResult.java
index dbb93ceaa..a2a19f5fc 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/search/SearchResult.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/search/SearchResult.java
@@ -35,6 +35,7 @@ public interface SearchResult extends 
org.apache.wiki.api.search.SearchResult {
      *  
      *  @return the WikiPage object containing this result
      */
+    @Override
     WikiPage getPage();
 
 }
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/search/SearchResultComparator.java 
b/jspwiki-main/src/main/java/org/apache/wiki/search/SearchResultComparator.java
index cbc45183f..7a2a58f62 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/search/SearchResultComparator.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/search/SearchResultComparator.java
@@ -36,7 +36,8 @@ public class SearchResultComparator implements Comparator< 
SearchResult >, Seria
      *  
      *  {@inheritDoc}
      */
-    public int compare( final SearchResult s1, final SearchResult s2 ) {
+    @Override
+    public int compare(final SearchResult s1, final SearchResult s2 ) {
         // Bigger scores are first.
         int res = s2.getScore() - s1.getScore();
         if( res == 0 ) {
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/tags/CookieTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/CookieTag.java
index 607c6c75a..d41c3e647 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/CookieTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/CookieTag.java
@@ -177,6 +177,7 @@ public class CookieTag
     /**
      *  {@inheritDoc}
      */
+    @Override
     public void release()
     {
         m_name = m_item = m_var = m_value = m_clear = m_scope = null;
@@ -214,6 +215,7 @@ public class CookieTag
     /**
      *  {@inheritDoc}
      */
+    @Override
     public int doEndTag()
     {
         String out = null;
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/EditorIteratorInfo.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/EditorIteratorInfo.java
index ba0685209..1da79cad2 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/EditorIteratorInfo.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/EditorIteratorInfo.java
@@ -29,6 +29,7 @@ import javax.servlet.jsp.tagext.VariableInfo;
  */
 public class EditorIteratorInfo extends TagExtraInfo
 {
+    @Override
     public VariableInfo[] getVariableInfo(final TagData data)
     {
 
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/HistoryIteratorInfo.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/HistoryIteratorInfo.java
index 1afa6ca12..374902736 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/HistoryIteratorInfo.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/HistoryIteratorInfo.java
@@ -29,7 +29,8 @@ import javax.servlet.jsp.tagext.VariableInfo;
  */
 public class HistoryIteratorInfo extends TagExtraInfo
 {
-    public VariableInfo[] getVariableInfo( final TagData data)
+    @Override
+    public VariableInfo[] getVariableInfo(final TagData data)
     {
 
         return new VariableInfo[]{ new VariableInfo( 
data.getAttributeString("id"),
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/IfNoSearchResultsTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/IfNoSearchResultsTag.java
index 334f021ed..1215fb5a2 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/IfNoSearchResultsTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/IfNoSearchResultsTag.java
@@ -31,6 +31,7 @@ import javax.servlet.jsp.PageContext;
 public class IfNoSearchResultsTag extends WikiTagBase {
     private static final long serialVersionUID = 0L;
     
+    @Override
     public final int doWikiStartTag() throws IOException {
         final Collection< ? > list = (Collection< ? 
>)pageContext.getAttribute( "searchresults", PageContext.REQUEST_SCOPE );
         if( list == null || list.size() == 0 ) {
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/IncludeResourcesTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/IncludeResourcesTag.java
index 88c4cde21..b61fa049c 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/IncludeResourcesTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/IncludeResourcesTag.java
@@ -34,6 +34,7 @@ public class IncludeResourcesTag extends WikiTagBase
         
     private String m_type;
 
+    @Override
     public void initTag()
     {
         super.initTag();
@@ -45,6 +46,7 @@ public class IncludeResourcesTag extends WikiTagBase
         m_type = type;
     }
     
+    @Override
     public int doWikiStartTag() throws Exception
     {
         //String marker = 
m_wikiContext.getEngine().getTemplateManager().getMarker(m_wikiContext, m_type);
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/LinkToParentTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/LinkToParentTag.java
index 2e59104f7..6833a89de 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/LinkToParentTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/LinkToParentTag.java
@@ -38,6 +38,7 @@ public class LinkToParentTag extends LinkToTag {
 
     private static final long serialVersionUID = 0L;
     
+    @Override
     public int doWikiStartTag() throws IOException {
         final Page p = m_wikiContext.getPage();
 
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/NextVersionTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/NextVersionTag.java
index 407712a76..068c1cdd6 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/NextVersionTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/NextVersionTag.java
@@ -33,6 +33,7 @@ public class NextVersionTag extends WikiTagBase {
 
     private static final long serialVersionUID = 0L;
 
+    @Override
     public final int doWikiStartTag() throws IOException {
         final Page page = m_wikiContext.getPage();
         int version = page.getVersion();
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/tags/PageDateTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/PageDateTag.java
index 25c571bf6..5c5f006f9 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/PageDateTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/PageDateTag.java
@@ -48,6 +48,7 @@ public class PageDateTag extends WikiTagBase {
 
     private String m_format;
 
+    @Override
     public void initTag() {
         super.initTag();
         m_format = null;
@@ -61,6 +62,7 @@ public class PageDateTag extends WikiTagBase {
         m_format = arg;
     }
 
+    @Override
     public final int doWikiStartTag() throws IOException {
         final Page page = m_wikiContext.getPage();
         if( page != null ) {
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/tags/PageExistsTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/PageExistsTag.java
index 6f82ce54b..14cb7d7f9 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/PageExistsTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/PageExistsTag.java
@@ -35,6 +35,7 @@ public class PageExistsTag
 {
     private static final long serialVersionUID = 0L;
     
+    @Override
     public final int doWikiStartTag()
         throws IOException,
                ProviderException
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/tags/PageTypeTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/PageTypeTag.java
index dfd4153ae..f558d16b7 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/PageTypeTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/PageTypeTag.java
@@ -39,6 +39,7 @@ public class PageTypeTag extends WikiTagBase {
     
     private String m_type;
 
+    @Override
     public void initTag() {
         super.initTag();
         m_type = null;
@@ -49,6 +50,7 @@ public class PageTypeTag extends WikiTagBase {
         m_type = arg.toLowerCase();
     }
 
+    @Override
     public final int doWikiStartTag() throws IOException {
         final Page page = m_wikiContext.getPage();
         if( page != null ) {
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/PageVersionTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/PageVersionTag.java
index 85c58cf8e..5b79aa2ae 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/PageVersionTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/PageVersionTag.java
@@ -32,6 +32,7 @@ public class PageVersionTag extends WikiTagBase {
 
     private static final long serialVersionUID = 0L;
 
+    @Override
     public final int doWikiStartTag() throws IOException {
         final Page page = m_wikiContext.getPage();
         if( page != null ) {
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultIteratorInfo.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultIteratorInfo.java
index fbdfe33c2..4ee2e3f27 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultIteratorInfo.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultIteratorInfo.java
@@ -30,7 +30,8 @@ import javax.servlet.jsp.tagext.VariableInfo;
  */
 public class SearchResultIteratorInfo extends TagExtraInfo {
 
-    public VariableInfo[] getVariableInfo( final TagData data ) {
+    @Override
+    public VariableInfo[] getVariableInfo(final TagData data ) {
 
         return new VariableInfo[]{ new VariableInfo( data.getAttributeString( 
"id" ),
                                                       
"org.apache.wiki.api.search.SearchResult",
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultsSizeTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultsSizeTag.java
index e966ce815..a7095f80d 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultsSizeTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultsSizeTag.java
@@ -33,6 +33,7 @@ public class SearchResultsSizeTag
 {
     private static final long serialVersionUID = 0L;
     
+    @Override
     public final int doWikiStartTag() throws IOException {
         final Collection< ? > list = ( Collection< ? > 
)pageContext.getAttribute( "searchresults", PageContext.REQUEST_SCOPE );
         if( list != null ) {
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultsTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultsTag.java
index 7b5a150ef..96ac7ef7b 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultsTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/SearchResultsTag.java
@@ -31,6 +31,7 @@ import javax.servlet.jsp.PageContext;
 public class SearchResultsTag extends WikiTagBase {
     private static final long serialVersionUID = 0L;
     
+    @Override
     public final int doWikiStartTag() throws IOException {
         final Collection< ? > list = ( Collection< ? > 
)pageContext.getAttribute( "searchresults", PageContext.REQUEST_SCOPE );
         
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/SetPaginationTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/SetPaginationTag.java
index 6a3b2d3fd..6b3e105ea 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/SetPaginationTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/SetPaginationTag.java
@@ -69,6 +69,7 @@ public class SetPaginationTag
     private String m_href;
     private String m_onclick;
 
+    @Override
     public void initTag()
     {
         super.initTag();
@@ -122,6 +123,7 @@ public class SetPaginationTag
     // 0 20 40 60 80 next last
     // first previous 20 40 *60* 80 100 next last
     // fist previous 40 60 80 100 120
+    @Override
     public int doWikiStartTag()
         throws IOException
     {
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/tags/TabTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/TabTag.java
index 92fdccea8..e93760e8a 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/TabTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/TabTag.java
@@ -56,6 +56,7 @@ public class TabTag extends WikiTagBase
     /**
      * {@inheritDoc}
      */
+    @Override
     public void doFinally()
     {
         super.doFinally();
@@ -109,6 +110,7 @@ public class TabTag extends WikiTagBase
     /**
      * {@inheritDoc}
      */
+    @Override
     public int doWikiStartTag() throws JspTagException
     {
         final TabbedSectionTag parent=(TabbedSectionTag)findAncestorWithClass( 
this, TabbedSectionTag.class );
@@ -156,6 +158,7 @@ public class TabTag extends WikiTagBase
     /**
      * {@inheritDoc}
      */
+    @Override
     public int doEndTag() throws JspTagException
     {
         final TabbedSectionTag parent=(TabbedSectionTag)findAncestorWithClass( 
this, TabbedSectionTag.class );
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/tags/TemplateDirTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/TemplateDirTag.java
index a215064fe..9c19f48d1 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/TemplateDirTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/TemplateDirTag.java
@@ -31,6 +31,7 @@ public class TemplateDirTag
 {
     private static final long serialVersionUID = 0L;
     
+    @Override
     public final int doWikiStartTag()
         throws IOException
     {
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiBodyTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiBodyTag.java
index dc6a2fef1..39bafbd24 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiBodyTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiBodyTag.java
@@ -40,6 +40,7 @@ public abstract class WikiBodyTag extends BodyTagSupport 
implements TryCatchFina
        protected WikiContext m_wikiContext;
     private static final Logger LOG = LogManager.getLogger( WikiBodyTag.class 
);
 
+    @Override
     public int doStartTag() throws JspException {
         try {
             m_wikiContext = (WikiContext) pageContext.getAttribute( 
Context.ATTR_CONTEXT, PageContext.REQUEST_SCOPE );
@@ -63,9 +64,11 @@ public abstract class WikiBodyTag extends BodyTagSupport 
implements TryCatchFina
      */
     public abstract int doWikiStartTag() throws JspException, IOException;
 
-    public void doCatch( final Throwable arg0) throws Throwable {
+    @Override
+    public void doCatch(final Throwable arg0) throws Throwable {
     }
 
+    @Override
     public void doFinally()
     {
         m_wikiContext = null;
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiLinkTag.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiLinkTag.java
index 7b5bb075d..5a90fbd14 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiLinkTag.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiLinkTag.java
@@ -47,7 +47,8 @@ public abstract class WikiLinkTag extends WikiTagBase {
     protected String m_template;
 
     
-    public void initTag() 
+    @Override
+    public void initTag()
     {
         super.initTag();
         m_pageName = m_template = null;
@@ -87,6 +88,7 @@ public abstract class WikiLinkTag extends WikiTagBase {
         }
     }
 
+    @Override
     public int doEndTag()
     {
         try
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiTagBase.java 
b/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiTagBase.java
index e282bf0ca..8eeb701f4 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiTagBase.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/tags/WikiTagBase.java
@@ -47,6 +47,7 @@ public abstract class WikiTagBase extends TagSupport 
implements TryCatchFinally
      * This method calls the parent setPageContext() but it also provides a 
way for a tag to initialize itself before
      * any of the setXXX() methods are called.
      */
+    @Override
     public void setPageContext( final PageContext arg0 ) {
         super.setPageContext( arg0 );
         initTag();
@@ -85,6 +86,7 @@ public abstract class WikiTagBase extends TagSupport 
implements TryCatchFinally
         return EVAL_PAGE;
     }
 
+    @Override
     public void doCatch( final Throwable th ) throws Throwable {
        LOG.error( th.getMessage(), th );
     }
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/ui/AbstractCommand.java 
b/jspwiki-main/src/main/java/org/apache/wiki/ui/AbstractCommand.java
index 869de6bed..922fbb47e 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/ui/AbstractCommand.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/ui/AbstractCommand.java
@@ -102,11 +102,13 @@ public abstract class AbstractCommand implements Command {
     /**
      * @see org.apache.wiki.api.core.Command#targetedCommand(Object)
      */
-    public abstract Command targetedCommand( final Object target );
+    @Override
+    public abstract Command targetedCommand(final Object target );
 
     /**
      * @see org.apache.wiki.api.core.Command#getContentTemplate()
      */
+    @Override
     public final String getContentTemplate() {
         return m_contentTemplate;
     }
@@ -114,6 +116,7 @@ public abstract class AbstractCommand implements Command {
     /**
      * @see org.apache.wiki.api.core.Command#getJSP()
      */
+    @Override
     public final String getJSP() {
         return m_jsp;
     }
@@ -121,11 +124,13 @@ public abstract class AbstractCommand implements Command {
     /**
      * @see org.apache.wiki.api.core.Command#getName()
      */
+    @Override
     public abstract String getName();
 
     /**
      * @see org.apache.wiki.api.core.Command#getRequestContext()
      */
+    @Override
     public final String getRequestContext() {
         return m_requestContext;
     }
@@ -133,6 +138,7 @@ public abstract class AbstractCommand implements Command {
     /**
      * @see org.apache.wiki.api.core.Command#getTarget()
      */
+    @Override
     public final Object getTarget() {
         return m_target;
     }
@@ -140,6 +146,7 @@ public abstract class AbstractCommand implements Command {
     /**
      * @see org.apache.wiki.api.core.Command#getURLPattern()
      */
+    @Override
     public final String getURLPattern() {
         return m_urlPattern;
     }
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/ui/GroupCommand.java 
b/jspwiki-main/src/main/java/org/apache/wiki/ui/GroupCommand.java
index ef390b88d..df0a1ce5e 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/ui/GroupCommand.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/ui/GroupCommand.java
@@ -92,7 +92,8 @@ public final class GroupCommand extends AbstractCommand {
      * @param target the GroupPrincipal to combine into the current Command
      * @return the new, targeted command
      */
-    public Command targetedCommand( final Object target ) {
+    @Override
+    public Command targetedCommand(final Object target ) {
         if( !( target instanceof GroupPrincipal ) ) {
             throw new IllegalArgumentException( "Target must non-null and of 
type GroupPrincipal." );
         }
@@ -105,6 +106,7 @@ public final class GroupCommand extends AbstractCommand {
      * @return the name
      * @see org.apache.wiki.api.core.Command#getName()
      */
+    @Override
     public String getName() {
         final Object target = getTarget();
         if ( target == null ) {
@@ -119,6 +121,7 @@ public final class GroupCommand extends AbstractCommand {
      * @return the permission
      * @see org.apache.wiki.api.core.Command#requiredPermission()
      */
+    @Override
     public Permission requiredPermission() {
         return m_permission;
     }
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/ui/PageCommand.java 
b/jspwiki-main/src/main/java/org/apache/wiki/ui/PageCommand.java
index 36b3a5d62..9dbcdfeea 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/ui/PageCommand.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/ui/PageCommand.java
@@ -99,7 +99,8 @@ public final class PageCommand extends AbstractCommand {
      * @return the new targeted command
      * @throws IllegalArgumentException if the target is not of the correct 
type
      */
-    public Command targetedCommand( final Object target ) {
+    @Override
+    public Command targetedCommand(final Object target ) {
         if( !( target instanceof Page ) ) {
             throw new IllegalArgumentException( "Target must non-null and of 
type Page." );
         }
@@ -109,6 +110,7 @@ public final class PageCommand extends AbstractCommand {
     /**
      * @see org.apache.wiki.api.core.Command#getName()
      */
+    @Override
     public String getName() {
         final Object target = getTarget();
         if( target == null ) {
@@ -120,6 +122,7 @@ public final class PageCommand extends AbstractCommand {
     /**
      * @see org.apache.wiki.api.core.Command#requiredPermission()
      */
+    @Override
     public Permission requiredPermission() {
         return m_permission;
     }
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/ui/RedirectCommand.java 
b/jspwiki-main/src/main/java/org/apache/wiki/ui/RedirectCommand.java
index bccb1e9db..e6125c330 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/ui/RedirectCommand.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/ui/RedirectCommand.java
@@ -68,7 +68,8 @@ public final class RedirectCommand extends AbstractCommand {
      * @param target the object to combine
      * @throws IllegalArgumentException if the target is not of the correct 
type
      */
-    public Command targetedCommand( final Object target ) {
+    @Override
+    public Command targetedCommand(final Object target ) {
         if ( !( target instanceof String ) ) {
             throw new IllegalArgumentException( "Target must non-null and of 
type String." );
         }
@@ -78,6 +79,7 @@ public final class RedirectCommand extends AbstractCommand {
     /**
      * @see org.apache.wiki.api.core.Command#getName()
      */
+    @Override
     public String getName() {
         final Object target = getTarget();
         if ( target == null ) {
@@ -91,6 +93,7 @@ public final class RedirectCommand extends AbstractCommand {
      *
      * @see org.apache.wiki.api.core.Command#requiredPermission()
      */
+    @Override
     public Permission requiredPermission() {
         return null;
     }
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/ui/progress/DefaultProgressManager.java
 
b/jspwiki-main/src/main/java/org/apache/wiki/ui/progress/DefaultProgressManager.java
index 3ca3d1fa8..be14611f1 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/ui/progress/DefaultProgressManager.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/ui/progress/DefaultProgressManager.java
@@ -69,6 +69,7 @@ public class DefaultProgressManager implements 
ProgressManager {
      *  @param pi ProgressItem to start
      *  @param id The progress identifier
      */
+    @Override
     public void startProgress( final ProgressItem pi, final String id ) {
         LOG.debug( "Adding " + id + " to progress queue" );
         m_progressingTasks.put( id, pi );
@@ -81,6 +82,7 @@ public class DefaultProgressManager implements 
ProgressManager {
      *
      *  @param id The progress identifier
      */
+    @Override
     public void stopProgress( final String id ) {
         LOG.debug( "Removed " + id + " from progress queue" );
         final ProgressItem pi = m_progressingTasks.remove( id );
@@ -96,6 +98,7 @@ public class DefaultProgressManager implements 
ProgressManager {
      *  @return a value between 0 to 100 indicating the progress
      *  @throws IllegalArgumentException If no such progress item exists.
      */
+    @Override
     public int getProgress( final String id ) throws IllegalArgumentException {
         final ProgressItem pi = m_progressingTasks.get( id );
         if( pi != null ) {
@@ -120,11 +123,13 @@ public class DefaultProgressManager implements 
ProgressManager {
         {
             return DefaultProgressManager.this.getProgress( progressId );
         }
-        
+
+        @Override
         public String getServletMapping() {
                return JSON_PROGRESSTRACKER;
         }
-        
+
+        @Override
         public void service( final HttpServletRequest req,
                              final HttpServletResponse resp,
                              final String actionName,
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/workflow/AbstractStep.java 
b/jspwiki-main/src/main/java/org/apache/wiki/workflow/AbstractStep.java
index acf44318d..3f42bebf6 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/workflow/AbstractStep.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/workflow/AbstractStep.java
@@ -92,13 +92,15 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
-    public final void addSuccessor( final Outcome outcome, final Step step ) {
+    @Override
+    public final void addSuccessor(final Outcome outcome, final Step step ) {
         m_successors.put( outcome, step );
     }
 
     /**
      * {@inheritDoc}
      */
+    @Override
     public final Collection< Outcome > getAvailableOutcomes() {
         final Set< Outcome > outcomes = m_successors.keySet();
         return Collections.unmodifiableCollection( outcomes );
@@ -107,6 +109,7 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
+    @Override
     public final List< String > getErrors() {
         return Collections.unmodifiableList( m_errors );
     }
@@ -114,16 +117,19 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
-    public abstract Outcome execute( Context ctx ) throws WikiException;
+    @Override
+    public abstract Outcome execute(Context ctx ) throws WikiException;
 
     /**
      * {@inheritDoc}
      */
+    @Override
     public abstract Principal getActor();
 
     /**
      * {@inheritDoc}
      */
+    @Override
     public final Date getEndTime() {
         return m_end;
     }
@@ -131,6 +137,7 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
+    @Override
     public final String getMessageKey() {
         return m_key;
     }
@@ -138,6 +145,7 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
+    @Override
     public final synchronized Outcome getOutcome() {
         return m_outcome;
     }
@@ -145,6 +153,7 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
+    @Override
     public final Date getStartTime() {
         return m_start;
     }
@@ -152,6 +161,7 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
+    @Override
     public final boolean isCompleted() {
         return m_completed;
     }
@@ -159,6 +169,7 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
+    @Override
     public final boolean isStarted() {
         return m_started;
     }
@@ -166,7 +177,8 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
-    public final synchronized void setOutcome( final Outcome outcome ) {
+    @Override
+    public final synchronized void setOutcome(final Outcome outcome ) {
         // Is this an allowed Outcome?
         if( !m_successors.containsKey( outcome ) ) {
             if( !Outcome.STEP_CONTINUE.equals( outcome ) && 
!Outcome.STEP_ABORT.equals( outcome ) ) {
@@ -188,6 +200,7 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
+    @Override
     public final synchronized void start() throws WikiException {
         if( m_started ) {
             throw new IllegalStateException( "Step already started." );
@@ -199,7 +212,8 @@ public abstract class AbstractStep implements Step {
     /**
      * {@inheritDoc}
      */
-    public final Step getSuccessor( final Outcome outcome ) {
+    @Override
+    public final Step getSuccessor(final Outcome outcome ) {
         return m_successors.get( outcome );
     }
 
@@ -211,7 +225,8 @@ public abstract class AbstractStep implements Step {
      * @param workflowId the parent workflow id to set
      * @param workflowContext the parent workflow context to set
      */
-    public final synchronized void setWorkflow( final int workflowId, final 
Map< String, Serializable > workflowContext ) {
+    @Override
+    public final synchronized void setWorkflow(final int workflowId, final 
Map< String, Serializable > workflowContext ) {
         this.workflowId = workflowId;
         this.workflowContext = workflowContext;
     }
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/workflow/Decision.java 
b/jspwiki-main/src/main/java/org/apache/wiki/workflow/Decision.java
index 6777f2876..7f833ecc5 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/workflow/Decision.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/workflow/Decision.java
@@ -121,7 +121,8 @@ public abstract class Decision extends AbstractStep {
      * @return the Outcome of the execution
      * @throws WikiException never
      */
-    public Outcome execute( final Context context ) throws WikiException {
+    @Override
+    public Outcome execute(final Context context ) throws WikiException {
         if( getOutcome().isCompletion() ) {
             return getOutcome();
         }
@@ -136,6 +137,7 @@ public abstract class Decision extends AbstractStep {
     /**
      * {@inheritDoc}
      */
+    @Override
     public final Principal getActor() {
         return m_actor;
     }
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/workflow/SimpleNotification.java 
b/jspwiki-main/src/main/java/org/apache/wiki/workflow/SimpleNotification.java
index 29cb4b2ee..6cf383f8e 100644
--- 
a/jspwiki-main/src/main/java/org/apache/wiki/workflow/SimpleNotification.java
+++ 
b/jspwiki-main/src/main/java/org/apache/wiki/workflow/SimpleNotification.java
@@ -63,6 +63,7 @@ public final class SimpleNotification extends Decision {
      *
      * @return <code>false</code> always.
      */
+    @Override
     public boolean isReassignable() {
         return false;
     }
diff --git 
a/jspwiki-main/src/main/java/org/apache/wiki/workflow/SystemPrincipal.java 
b/jspwiki-main/src/main/java/org/apache/wiki/workflow/SystemPrincipal.java
index 4de73ae41..8e75b0008 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/workflow/SystemPrincipal.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/workflow/SystemPrincipal.java
@@ -43,6 +43,7 @@ public final class SystemPrincipal implements Principal {
     /**
      * {@inheritDoc}
      */
+    @Override
     public String getName() {
         return m_name;
     }
diff --git a/jspwiki-main/src/main/java/org/apache/wiki/workflow/Task.java 
b/jspwiki-main/src/main/java/org/apache/wiki/workflow/Task.java
index cfe33985c..87a68400b 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/workflow/Task.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/workflow/Task.java
@@ -69,6 +69,7 @@ public abstract class Task extends AbstractStep {
      *
      * @return the system principal
      */
+    @Override
     public final Principal getActor() {
         return SystemPrincipal.SYSTEM_USER;
     }
diff --git a/jspwiki-main/src/test/java/SamplePlugin2.java 
b/jspwiki-main/src/test/java/SamplePlugin2.java
index 5ea6d32c3..e188ddc05 100644
--- a/jspwiki-main/src/test/java/SamplePlugin2.java
+++ b/jspwiki-main/src/test/java/SamplePlugin2.java
@@ -30,7 +30,8 @@ public class SamplePlugin2 implements Plugin {
     public void initialize( final Engine engine ) throws PluginException {
     }
 
-    public String execute( final Context context, final Map< String, String > 
params ) throws PluginException {
+    @Override
+    public String execute(final Context context, final Map< String, String > 
params ) throws PluginException {
         return params.get( "text" );
     }
 
diff --git 
a/jspwiki-main/src/test/java/org/apache/wiki/auth/SecurityEventTrap.java 
b/jspwiki-main/src/test/java/org/apache/wiki/auth/SecurityEventTrap.java
index c52102d56..21c2db494 100644
--- a/jspwiki-main/src/test/java/org/apache/wiki/auth/SecurityEventTrap.java
+++ b/jspwiki-main/src/test/java/org/apache/wiki/auth/SecurityEventTrap.java
@@ -34,6 +34,7 @@ public class SecurityEventTrap implements WikiEventListener
     private WikiSecurityEvent m_lastEvent;
     private final List<WikiSecurityEvent> m_events    = new 
ArrayList<WikiSecurityEvent>();
 
+    @Override
     public void actionPerformed(final WikiEvent event )
     {
         if ( event instanceof WikiSecurityEvent )
diff --git 
a/jspwiki-main/src/test/java/org/apache/wiki/plugin/SamplePlugin3.java 
b/jspwiki-main/src/test/java/org/apache/wiki/plugin/SamplePlugin3.java
index f7afdcaad..4d41f7ef5 100644
--- a/jspwiki-main/src/test/java/org/apache/wiki/plugin/SamplePlugin3.java
+++ b/jspwiki-main/src/test/java/org/apache/wiki/plugin/SamplePlugin3.java
@@ -36,7 +36,8 @@ public class SamplePlugin3 implements Plugin {
     public void initialize( final Engine engine ) throws PluginException {
     }
 
-    public String execute( final Context context, final Map< String, String > 
params ) throws PluginException {
+    @Override
+    public String execute(final Context context, final Map< String, String > 
params ) throws PluginException {
         return params.get( "text" );
     }
 
diff --git a/jspwiki-main/src/test/java/org/apache/wiki/web/TestContainer.java 
b/jspwiki-main/src/test/java/org/apache/wiki/web/TestContainer.java
index 931ba8a1f..70a8d316d 100644
--- a/jspwiki-main/src/test/java/org/apache/wiki/web/TestContainer.java
+++ b/jspwiki-main/src/test/java/org/apache/wiki/web/TestContainer.java
@@ -283,6 +283,7 @@ public class TestContainer {
      */
     public static final class ShutdownHandler extends AbstractHandler {
 
+        @Override
         public void handle(final String target, final Request baseRequest, 
final HttpServletRequest request, final HttpServletResponse response ) throws 
IOException, ServletException {
             if( request.getRequestURI().contains( SHUTDOWN_CMD ) ) {
                 LOG.error( "stop cmd received, shutting down server" );
diff --git a/jspwiki-main/src/test/java/org/apache/wiki/workflow/TaskTest.java 
b/jspwiki-main/src/test/java/org/apache/wiki/workflow/TaskTest.java
index dcbe890e2..fe2bdf0e1 100644
--- a/jspwiki-main/src/test/java/org/apache/wiki/workflow/TaskTest.java
+++ b/jspwiki-main/src/test/java/org/apache/wiki/workflow/TaskTest.java
@@ -47,7 +47,8 @@ public class TaskTest {
             super( workflow.getId(), workflow.getAttributes(), "task.normal" );
         }
 
-        public Outcome execute( final Context context ) {
+        @Override
+        public Outcome execute(final Context context ) {
             return Outcome.STEP_COMPLETE;
         }
 
@@ -62,7 +63,8 @@ public class TaskTest {
             super( workflow.getId(), workflow.getAttributes(), "task.error" );
         }
 
-        public Outcome execute( final Context context ) {
+        @Override
+        public Outcome execute(final Context context ) {
             addError( "Found an error." );
             addError( "Found a second one!" );
             return Outcome.STEP_ABORT;
diff --git a/jspwiki-util/src/main/java/org/apache/wiki/util/MailUtil.java 
b/jspwiki-util/src/main/java/org/apache/wiki/util/MailUtil.java
index b0c29cd4f..3a3720c54 100644
--- a/jspwiki-util/src/main/java/org/apache/wiki/util/MailUtil.java
+++ b/jspwiki-util/src/main/java/org/apache/wiki/util/MailUtil.java
@@ -445,6 +445,7 @@ public final class MailUtil {
          * Returns the password used to authenticate to the SMTP server.
          * @return <code>PasswordAuthentication</code>
          */
+        @Override
         public PasswordAuthentication getPasswordAuthentication()
         {
             if ( BLANK.equals(m_pass) )
diff --git a/jspwiki-util/src/main/java/org/apache/wiki/util/PriorityList.java 
b/jspwiki-util/src/main/java/org/apache/wiki/util/PriorityList.java
index 87c2ffd4a..3e847e805 100644
--- a/jspwiki-util/src/main/java/org/apache/wiki/util/PriorityList.java
+++ b/jspwiki-util/src/main/java/org/apache/wiki/util/PriorityList.java
@@ -75,6 +75,7 @@ public class PriorityList<E>
      *  @param o Object to add.
      *  @return true, as per the general Collections.add contract.
      */
+    @Override
     public boolean add(final E o )
     {
         add( o, DEFAULT_PRIORITY );
@@ -88,6 +89,7 @@ public class PriorityList<E>
      *  @param index The index.
      *  @return The object at the list at the position "index".
      */
+    @Override
     public E get(final int index )
     {
         return m_elements.get( index ).m_object;
@@ -98,6 +100,7 @@ public class PriorityList<E>
      *  
      *  @return size of the list.
      */
+    @Override
     public int size()
     {
         return m_elements.size();
diff --git 
a/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/CollatorComparator.java
 
b/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/CollatorComparator.java
index 94f44f6f2..cdc77462a 100644
--- 
a/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/CollatorComparator.java
+++ 
b/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/CollatorComparator.java
@@ -61,6 +61,7 @@ public class CollatorComparator implements Comparator<String>
      * (non-Javadoc)
      * @see java.util.Comparator#compare(java.lang.Object, java.lang.Object)
      */
+    @Override
     public int compare(final String str1, final String str2 )
     {
         if( StringUtils.equals( str1, str2 ) ) {
diff --git 
a/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/HumanComparator.java
 
b/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/HumanComparator.java
index 4baac9ca7..d1ea2787f 100644
--- 
a/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/HumanComparator.java
+++ 
b/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/HumanComparator.java
@@ -78,6 +78,7 @@ public class HumanComparator implements Comparator< String > {
      * (non-Javadoc)
      * @see java.util.Comparator#compare(java.lang.Object, java.lang.Object)
      */
+    @Override
     public int compare(final String str1, final String str2 )
     {
         // Some quick and easy checks
diff --git 
a/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/JavaNaturalComparator.java
 
b/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/JavaNaturalComparator.java
index e57feedc9..687d9d57d 100644
--- 
a/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/JavaNaturalComparator.java
+++ 
b/jspwiki-util/src/main/java/org/apache/wiki/util/comparators/JavaNaturalComparator.java
@@ -36,6 +36,7 @@ public class JavaNaturalComparator implements 
Comparator<String>
      * (non-Javadoc)
      * @see java.util.Comparator#compare(java.lang.Object, java.lang.Object)
      */
+    @Override
     public int compare(final String str1, final String str2 ) {
         if (StringUtils.equals( str1, str2 ) ) {
                return 0; // the same object

Reply via email to