Task #10372 (closed)
CheckedPath should not expose File instances
| Reported by: | mtbcarroll | Owned by: | mtbcarroll |
|---|---|---|---|
| Priority: | minor | Milestone: | 5.0.0-beta1 |
| Component: | OmeroFs | Version: | n.a. |
| Keywords: | FS | Cc: | fs@… |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | 0.0d |
| Sprint: | n.a. |
Description
As few classes as possible should be handling Java File instances corresponding to anything within the managed repository. FileBuffers and suchlike can be passed around instead so that the local filesystem view of the path is not accidentally used and platform-specific details are not a factor for most methods. Investigate refactoring CheckedPath, RawFileBean, ManagedRepositoryI, etc. accordingly.
Change History (3)
comment:1 Changed 6 years ago by mtbcarroll
- Status changed from new to accepted
comment:2 Changed 6 years ago by mtbcarroll
- Resolution set to fixed
- Status changed from accepted to closed
comment:3 Changed 6 years ago by Mark Carroll <m.t.b.carroll@…>
- Remaining Time set to 0
(In [a33f4c0a30635c377fa23fa0ce119fcb3e6989c7/ome.git] on branch develop) Fix #10372: hide CheckedPath?'s File instance
Note: See
TracTickets for help on using
tickets.
You may also have a look at Agilo extensions to the ticket.
Fixed by https://github.com/openmicroscopy/openmicroscopy/pull/881