[MAGNOLIA-2516] Cleanup/consolidate i.m.cms.beans.runtime.File and the various Document classes Created: 16/Dec/08 Updated: 19/Dec/16 Resolved: 04/Nov/15 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major |
| Reporter: | Magnolia International | Assignee: | Philipp Bärfuss |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||||||
| Issue Links: |
|
||||||||||||||||
| Template: |
|
||||||||||||||||
| Patch included: |
Yes
|
||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||
| Date of First Response: | |||||||||||||||||
| Description |
|
info.magnolia.cms.beans.runtime.File, info.magnolia.cms.beans.runtime.FileProperties, info.magnolia.cms.beans.runtime.Document and even info.magnolia.module.dms.beans.Document all seem to have a fairly similar purpose. They should be cleaned up, given better name(s) and packages. |
| Comments |
| Comment by Luca Boati [ 27/Aug/09 ] |
|
Maybe you are refactoring these classes, anyway... Saving a binary file, info.magnolia.module.admininterface.saveHandlerImpl set the contentType attribute with key FileProperties.PROPERTY_CONTENTTYPE:
Into info.magnolia.cms.beans.runtime.File the constructor set the contentType property getting the value of key "contentType" from nodeData :
fixing with |
| Comment by Michael Mühlebach [ 04/Nov/15 ] |
|
Given the thousands of other issues we have open that are more highly requested, we won't be able to address this issue in the foreseeable future. Instead we will focus on issues with a higher impact, and more votes. |