Skip to content

Create Pref Service #102

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

Merged
hinerm merged 16 commits into from
Jul 25, 2014
Merged

Create Pref Service #102

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
9f210a7
Refactor Prefs to PrefService
hinerm Jul 18, 2014
4ef6f00
Update Prefs usage to PrefService
hinerm Jul 21, 2014
992f0b8
ModuleService: add save/load item methods
hinerm Jul 21, 2014
cc97d1b
Mark saveValue and loadValue deprecated
hinerm Jul 21, 2014
d503e31
Refactor for ModuleService saving and loading
hinerm Jul 21, 2014
ce0f105
AbstractPrefService: account for priority
hinerm Jul 21, 2014
9050328
Clean up code
hinerm Jul 21, 2014
6fe0553
DefaultPrefService: add TODO
hinerm Jul 21, 2014
f857d0f
Deprecate Prefs
hinerm Jul 21, 2014
a9a109c
PrefService: remove setStaticBehavior method
hinerm Jul 24, 2014
9abe3d9
PrefService: remove Preferences imports
hinerm Jul 24, 2014
7ff590f
Rename preferences to prefs package
hinerm Jul 25, 2014
344d6bc
DefaultPrefService: fix javadoc
ctrueden Jul 25, 2014
963d4eb
DefaultPrefService: do not reference Prefs class
ctrueden Jul 25, 2014
cdeb735
Add down-the-middle unit tests for PrefService
ctrueden Jul 25, 2014
a0c6c95
AbstractPrefService: suppress warnings
hinerm Jul 25, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Mark saveValue and loadValue deprecated 
ModuleItem#saveValue and ModuleItem#loadValue are now deprecated in
favor of ModuleService methods, which can be called from a Contextual
environment and thus use the new PrefService API.
  • Loading branch information
@hinerm
hinerm committed Jul 25, 2014
commit cc97d1bc05155913b4bc8860f69b17b47c0b651c
2 changes: 2 additions & 0 deletions src/main/java/org/scijava/module/AbstractModuleItem.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -137,6 +137,7 @@ public String getPersistKey() {
* value for null.
*/
@Override
@Deprecated
public T loadValue() {
// if there is nothing to load from persistence return nothing
if (!isPersisted()) return null;
Expand All @@ -157,6 +158,7 @@ public T loadValue() {
}

@Override
@Deprecated
public void saveValue(final T value) {
if (!isPersisted()) return;

Expand Down
8 changes: 8 additions & 0 deletions src/main/java/org/scijava/module/ModuleItem.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,13 +93,21 @@ public interface ModuleItem<T> extends BasicDetails {
* Note that this is different than obtaining a module instance's current
* value for the input; see {@link #getValue(Module)} for that.
* </p>
*
* @deprecated
* @see ModuleService#load(ModuleItem)
*/
@Deprecated
T loadValue();

/**
* Saves the given value to persistent storage. This allows later restoration
* of the value via {@link #loadValue()}, even from a different JVM.
*
* @deprecated
* @see ModuleService#save(ModuleItem, Object)
*/
@Deprecated
void saveValue(T value);

/** Gets the function that is called to initialize the item's value. */
Expand Down