Task #12259 (closed)
Opened 10 years ago
Closed 8 years ago
Review DBEnumCheck
Reported by: | mtbcarroll | Owned by: | jamoore |
---|---|---|---|
Priority: | minor | Milestone: | Unscheduled |
Component: | General | Version: | 5.0.1 |
Keywords: | n.a. | Cc: | mlinkert, ajpatterson, jamoore |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | n.a. |
Sprint: | n.a. |
Description
DBEnumCheck is old and crufty. Should the list of formats should be removed in favor of calling ImageReader.hasCompanionFiles() on the actual readers? Do we ever set Image.format to a companion setting anyway? This class should be reviewed. If is it to be removed, also delete its key from the configuration table in the database.
see https://github.com/openmicroscopy/openmicroscopy/pull/2406#issuecomment-42394499
Change History (5)
comment:1 Changed 10 years ago by jamoore
- Milestone changed from 5.0.3 to 5.1.0-m1
comment:2 Changed 10 years ago by jamoore
- Milestone changed from 5.1.0-m1 to 5.1.0-m2
To be done with final schema/DB changes.
comment:3 Changed 9 years ago by jamoore
- Milestone changed from 5.1.0-m2 to 5.x
Listed on trello with 5.2 DB changes. Pushing.
comment:4 Changed 8 years ago by jamoore
- Cc changed from mlinkert,ajpatterson,jamoore to mlinkert, ajpatterson, jamoore
- Milestone changed from 5.x to Unscheduled
comment:5 Changed 8 years ago by mtbcarroll
- Resolution set to fixed
- Status changed from new to closed
I don't see my getting to this for 5.0.3. Pushing to 5.1 m1, but we can always backport the fix to 5.0.4