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.

Sign up for GitHub

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

Jump to bottom

DefaultModuleService: change persisting of items #236

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

DefaultModuleService: change persisting of items #236

wants to merge 5 commits into from

Commits on Aug 8, 2016

  1. DefaultModuleService: change persisting of items 

    Disable the persisting of a ModuleItems if an initialize() method is set
and is not empty. This way, initialize() can overwrite the value of a
ModuleItem even if persist=true is assumed implicitly.

See:
* imagej/imagej-legacy#137 (comment)
    @stelfrich
    stelfrich committed Aug 8, 2016
    Configuration menu
    Copy the full SHA
    876e9bf View commit details
    Browse the repository at this point in the history

  2. Parameter: update initializer() documentation

    @stelfrich
    stelfrich committed Aug 8, 2016
    Configuration menu
    Copy the full SHA
    d725de5 View commit details
    Browse the repository at this point in the history

  3. Add test for persistance behavior of ModuleItems

    @stelfrich
    stelfrich committed Aug 8, 2016
    Configuration menu
    Copy the full SHA
    04e9442 View commit details
    Browse the repository at this point in the history

  4. ModuleServiceText: clean up

    @stelfrich
    stelfrich committed Aug 8, 2016
    Configuration menu
    Copy the full SHA
    44029b6 View commit details
    Browse the repository at this point in the history

  5. Add ItemPersistence to denote persistence behavior 

    Introduces the ItemPersistence enum (YES, NO, DEFAULT). YES and NO
overwrite additional checks (i.e. precedence of an initializer, value
is default value). In case of DEFAULT, the value is persisted only if
it is not the default value and no initializer is set.
    @stelfrich
    stelfrich committed Aug 8, 2016
    Configuration menu
    Copy the full SHA
    b4e8a32 View commit details
    Browse the repository at this point in the history