You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by gs...@apache.org on 2007/11/03 23:29:58 UTC
svn commit: r591700 -
/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java
Author: gseitz
Date: Sat Nov 3 15:29:53 2007
New Revision: 591700
URL: http://svn.apache.org/viewvc?rev=591700&view=rev
Log:
WICKET-1043: updated javadoc and remove Page#newVersionManager
Modified:
wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java
Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java?rev=591700&r1=591699&r2=591700&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java Sat Nov 3 15:29:53 2007
@@ -35,6 +35,7 @@
import org.apache.wicket.markup.html.form.Form;
import org.apache.wicket.model.IModel;
import org.apache.wicket.request.RequestParameters;
+import org.apache.wicket.session.ISessionStore;
import org.apache.wicket.session.pagemap.IPageMapEntry;
import org.apache.wicket.settings.IDebugSettings;
import org.apache.wicket.util.concurrent.ConcurrentHashMap;
@@ -99,10 +100,8 @@
*
* <li><b>Back Button </b>- Pages can support the back button by enabling versioning with a call to
* setVersioned(boolean). If a Page is versioned and changes occur to it which need to be tracked, a
- * version manager will be installed using the overridable factory method newVersionManager(). The
- * default version manager returned by the base implementation of this method is an instance of
- * UndoPageVersionManager, which manages versions of a page by keeping change records that can be
- * reversed at a later time.
+ * version manager will be installed using the {@link ISessionStore}'s factory method
+ * newVersionManager().
*
* <li><b>Security </b>- See {@link IAuthorizationStrategy},
* {@link SimplePageAuthorizationStrategy}
@@ -146,7 +145,7 @@
*
*/
public Page deserializePage(int id, String name, Page page, ObjectInputStream stream)
- throws IOException, ClassNotFoundException;
+ throws IOException, ClassNotFoundException;
/**
* Called from the {@link Page#writeObject(java.io.ObjectOutputStream)} method.
@@ -337,7 +336,7 @@
// that forwards to IAuthorizationStrategy for RequestListenerInterface
// invocations.
public void afterCallComponent(final Component component,
- final RequestListenerInterface listener)
+ final RequestListenerInterface listener)
{
}
@@ -358,7 +357,7 @@
// that forwards to IAuthorizationStrategy for RequestListenerInterface
// invocations.
public void beforeCallComponent(final Component component,
- final RequestListenerInterface listener)
+ final RequestListenerInterface listener)
{
}
@@ -380,7 +379,7 @@
if (renderedComponents.add(component) == false)
{
throw new MarkupException("The component " + component +
- " has the same wicket:id as another component already added at the same level");
+ " has the same wicket:id as another component already added at the same level");
}
if (log.isDebugEnabled())
{
@@ -585,7 +584,7 @@
else
{
log.info("No version manager available to retrieve requested versionNumber " +
- versionNumber);
+ versionNumber);
return null;
}
}
@@ -614,7 +613,7 @@
// If we went all the way back to the original page
if (page != null && page.getCurrentVersionNumber() == 0 &&
- page.getAjaxVersionNumber() == 0)
+ page.getAjaxVersionNumber() == 0)
{
// remove version info
page.versionManager = null;
@@ -655,7 +654,7 @@
levels++;
}
buffer.append(StringValue.repeat(levels, " ") + component.getPageRelativePath() +
- ":" + Classes.simpleName(component.getClass()));
+ ":" + Classes.simpleName(component.getClass()));
return null;
}
});
@@ -751,7 +750,7 @@
if (getStatelessHint())
{
log.warn("Page '" + this + "' is not stateless because it is not bookmarkable, " +
- "but the stateless hint is set to true!");
+ "but the stateless hint is set to true!");
}
}
@@ -783,7 +782,7 @@
if (!stateless.booleanValue() && getStatelessHint())
{
log.warn("Page '" + this + "' is not stateless because of '" + returnArray[0] +
- "' but the stateless hint is set to true!");
+ "' but the stateless hint is set to true!");
}
}
@@ -812,13 +811,13 @@
* will remain unchanged.
*/
public final void removePersistedFormData(final Class formClass,
- final boolean disablePersistence)
+ final boolean disablePersistence)
{
// Check that formClass is an instanceof Form
if (!Form.class.isAssignableFrom(formClass))
{
throw new WicketRuntimeException("Form class " + formClass.getName() +
- " is not a subclass of Form");
+ " is not a subclass of Form");
}
// Visit all children which are an instance of formClass
@@ -965,8 +964,8 @@
if (value && !isBookmarkable())
{
throw new WicketRuntimeException(
- "Can't set stateless hint to true on a page when the page is not bookmarkable, page: " +
- this);
+ "Can't set stateless hint to true on a page when the page is not bookmarkable, page: " +
+ this);
}
setFlag(FLAG_STATELESS_HINT, value);
}
@@ -994,13 +993,13 @@
if (versionManager != null)
{
return "[Page class = " + getClass().getName() + ", id = " + getId() + ", version = " +
- versionManager.getCurrentVersionNumber() + ", ajax = " +
- versionManager.getAjaxVersionNumber() + "]";
+ versionManager.getCurrentVersionNumber() + ", ajax = " +
+ versionManager.getAjaxVersionNumber() + "]";
}
else
{
return "[Page class = " + getClass().getName() + ", id = " + getId() + ", version = " +
- 0 + "]";
+ 0 + "]";
}
}
@@ -1035,9 +1034,9 @@
// Add to explanatory string to buffer
buffer.append(Integer.toString(unrenderedComponents.size()) + ". " +
- component + "\n");
+ component + "\n");
String metadata = (String)component
- .getMetaData(Component.CONSTRUCTED_AT_KEY);
+ .getMetaData(Component.CONSTRUCTED_AT_KEY);
if (metadata != null)
{
buffer.append(metadata);
@@ -1078,13 +1077,15 @@
{
Component sibling = (Component)iterator2.next();
if (!sibling.isVisible() && sibling instanceof MarkupContainer &&
- ((MarkupContainer)sibling).isTransparentResolver())
+ ((MarkupContainer)sibling).isTransparentResolver())
{
// we found a transparent container that isn't visible
// then ignore this component and only do a debug statement here.
- log.warn("Component " + component +
- " wasn't rendered but most likely it has a transparent parent: " +
- sibling);
+ log
+ .warn("Component " +
+ component +
+ " wasn't rendered but most likely it has a transparent parent: " +
+ sibling);
iterator.remove();
break;
}
@@ -1095,8 +1096,8 @@
{
// Throw exception
throw new WicketRuntimeException(
- "The component(s) below failed to render. A common problem is that you have added a component in code but forgot to reference it in the markup (thus the component will never be rendered).\n\n" +
- buffer.toString());
+ "The component(s) below failed to render. A common problem is that you have added a component in code but forgot to reference it in the markup (thus the component will never be rendered).\n\n" +
+ buffer.toString());
}
}
}
@@ -1205,7 +1206,7 @@
// Auto components do not participate in versioning since they are
// added during the rendering phase (which is normally illegal).
if (component.isAuto() || (parent == null && !component.isVersioned()) ||
- (parent != null && !parent.isVersioned()))
+ (parent != null && !parent.isVersioned()))
{
return false;
}
@@ -1331,7 +1332,7 @@
// Write out an xml declaration if the markup stream and settings allow
final MarkupStream markupStream = findMarkupStream();
if ((markupStream != null) && (markupStream.getXmlDeclaration() != null) &&
- (application.getMarkupSettings().getStripXmlDeclarationFromOutput() == false))
+ (application.getMarkupSettings().getStripXmlDeclarationFromOutput() == false))
{
response.write("<?xml version='1.0' encoding='");
response.write(encoding);
@@ -1377,14 +1378,6 @@
setPageMap(map);
setNextAvailableId();
- }
-
- /**
- * @return Factory method that creates a version manager for this Page
- */
- protected final IPageVersionManager newVersionManager()
- {
- return null;
}
/**