Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AjaxFlexibleFileUpload is not serializable #653

Closed
paulhoadley opened this issue May 21, 2015 · 0 comments
Closed

AjaxFlexibleFileUpload is not serializable #653

paulhoadley opened this issue May 21, 2015 · 0 comments

Comments

@paulhoadley
Copy link
Contributor

The Logger in AjaxFlexibleFileUpload is declared just protected final, and Logger is not serialisable. Declaring this private static final should be sufficient.

paulhoadley added a commit to paulhoadley/wonder that referenced this issue May 21, 2015
Here we just make the Logger static so that the class is serializable.
@darkv darkv closed this as completed May 26, 2015
elemgee added a commit to elemgee/wonder that referenced this issue Jul 22, 2015
* master: (26 commits)
  javadocs
  don't rely on notification to enable processing ec changes
  correctly look for existing class attribute
  exclude comment and doctype tags from appending class attribute
  use outline instead of border for styling
  add tests
  clean javadoc
  check for empty array
  fix default submit button for IE
  add serial version to AjaxProgress so it can be persisted
  recompile gluonj for Java 8 support
  Fix for a minor JS issue on ERModern login pages
  Removed superfluous setUp method as all this is handled in the superclass.
  Makes AjaxFlexibleFileUpload serializable. wocommunity#653
  * Added a dependency for the homegrown mockmail.jar used to mock SMTP tests in ERJavaMail. Thanks to @hprange for making the jar available on the repo!  * Enable tests in the global pom file and disable tests for ERAttachment and ERQuartzScheduler  * Add various missing dependencies that caused the build to fail  * Fix duplicate artifact ID  * Add ERModernMoviesTest to the Tests module  * Enable the Tests module to include ERModernMoviesTest in the build  * Fix project version in existing Tests module projects
  First cut of an integration-testing app, based on ERModernMoviesDemo.
  preserve complete URLs if context is set to
  generate complete URLs if secure mode changes
  Fixed Dutch LocalizableStrings so that it is UTF-16LE
  Fixed Localizable Strings for Dutch: some Gremlins caused havoc.
  ...
elemgee added a commit to elemgee/wonder that referenced this issue Jul 22, 2015
…display_cleanup

* develop: (27 commits)
  remove JRebel manifest
  javadocs
  don't rely on notification to enable processing ec changes
  correctly look for existing class attribute
  exclude comment and doctype tags from appending class attribute
  use outline instead of border for styling
  add tests
  clean javadoc
  check for empty array
  fix default submit button for IE
  add serial version to AjaxProgress so it can be persisted
  recompile gluonj for Java 8 support
  Fix for a minor JS issue on ERModern login pages
  Removed superfluous setUp method as all this is handled in the superclass.
  Makes AjaxFlexibleFileUpload serializable. wocommunity#653
  * Added a dependency for the homegrown mockmail.jar used to mock SMTP tests in ERJavaMail. Thanks to @hprange for making the jar available on the repo!  * Enable tests in the global pom file and disable tests for ERAttachment and ERQuartzScheduler  * Add various missing dependencies that caused the build to fail  * Fix duplicate artifact ID  * Add ERModernMoviesTest to the Tests module  * Enable the Tests module to include ERModernMoviesTest in the build  * Fix project version in existing Tests module projects
  First cut of an integration-testing app, based on ERModernMoviesDemo.
  preserve complete URLs if context is set to
  generate complete URLs if secure mode changes
  Fixed Dutch LocalizableStrings so that it is UTF-16LE
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants