Author: thrantal
Date: Sat Sep 6 13:48:21 2008
New Revision: 692733
URL: http://svn.apache.org/viewvc?rev=692733&view=rev
Log:
Added missing models to FileUpload fields. This should fix the build (see
http://wicketstuff.org/teamcity/viewLog.html?tab=buildResultsDiv&buildTypeId=bt2&buildId=1448
).
However, I'm not sure how FileUploadField should work exactly. Should the
constructor without a model throw an exception? Maybe not, because the model
can be added after construction as well with setModel, and the code like the
previous versions of these files should crash like it did.
- also added some missing Java 5 and Wicket 1.4 features in
MockPageWithFormAndUploadField
Modified:
wicket/trunk/wicket/src/test/java/org/apache/wicket/markup/html/form/upload/MockPageWithFormAndUploadField.java
wicket/trunk/wicket/src/test/java/org/apache/wicket/util/tester/MockFormFileUploadPage.java
Modified:
wicket/trunk/wicket/src/test/java/org/apache/wicket/markup/html/form/upload/MockPageWithFormAndUploadField.java
URL:
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/test/java/org/apache/wicket/markup/html/form/upload/MockPageWithFormAndUploadField.java?rev=692733&r1=692732&r2=692733&view=diff
==============================================================================
---
wicket/trunk/wicket/src/test/java/org/apache/wicket/markup/html/form/upload/MockPageWithFormAndUploadField.java
(original)
+++
wicket/trunk/wicket/src/test/java/org/apache/wicket/markup/html/form/upload/MockPageWithFormAndUploadField.java
Sat Sep 6 13:48:21 2008
@@ -18,6 +18,7 @@
import org.apache.wicket.markup.html.WebPage;
import org.apache.wicket.markup.html.form.Form;
+import org.apache.wicket.model.Model;
/**
* Mock page for use in the FileUploadFieldTest
@@ -50,7 +51,7 @@
fileUpload = fileUploadField.getFileUpload();
}
};
- fileUploadField = new FileUploadField("upload");
+ fileUploadField = new FileUploadField("upload", new
Model<FileUpload>());
form.add(fileUploadField);
add(form);
}
Modified:
wicket/trunk/wicket/src/test/java/org/apache/wicket/util/tester/MockFormFileUploadPage.java
URL:
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/test/java/org/apache/wicket/util/tester/MockFormFileUploadPage.java?rev=692733&r1=692732&r2=692733&view=diff
==============================================================================
---
wicket/trunk/wicket/src/test/java/org/apache/wicket/util/tester/MockFormFileUploadPage.java
(original)
+++
wicket/trunk/wicket/src/test/java/org/apache/wicket/util/tester/MockFormFileUploadPage.java
Sat Sep 6 13:48:21 2008
@@ -24,6 +24,7 @@
import org.apache.wicket.markup.html.form.upload.FileUpload;
import org.apache.wicket.markup.html.form.upload.FileUploadField;
import org.apache.wicket.model.CompoundPropertyModel;
+import org.apache.wicket.model.Model;
import org.apache.wicket.util.lang.Bytes;
/**
@@ -75,11 +76,12 @@
public MockFormFileUploadPage()
{
domainObject = new MockDomainObjectFileUpload();
- Form form = new Form("form", new
CompoundPropertyModel(domainObject))
+ Form<MockDomainObjectFileUpload> form = new
Form<MockDomainObjectFileUpload>("form", new
CompoundPropertyModel<MockDomainObjectFileUpload>(domainObject))
{
private static final long serialVersionUID = 1L;
- protected void onSubmit()
+ @Override
+ protected void onSubmit()
{
fileUpload = fileUploadField.getFileUpload();
}
@@ -87,8 +89,8 @@
add(form);
form.setMultiPart(true);
form.setMaxSize(Bytes.kilobytes(100));
- form.add(new TextField("text"));
- fileUploadField = new FileUploadField("file");
+ form.add(new TextField<String>("text"));
+ fileUploadField = new FileUploadField("file", new
Model<FileUpload>());
fileUploadField.setRequired(true);
form.add(fileUploadField);
}