Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

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

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

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 10 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
Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.68990 sec.)

We're Hiring!