[MGNLDMS-121] Icon general.png not displayed in dms tree Created: 06/Feb/08 Updated: 03/Jul/14 Resolved: 10/Sep/08 |
|
| Status: | Closed |
| Project: | Document Management System (closed) |
| Component/s: | None |
| Affects Version/s: | 1.2.3 |
| Fix Version/s: | 1.2.6 |
| Type: | Bug | Priority: | Major |
| Reporter: | Capitaine Harold | Assignee: | Philipp Bärfuss |
| Resolution: | Fixed | Votes: | 1 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Date of First Response: | |||||||||
| Description |
|
Go in the dms section, add a new document as name.sh. The file icon is not displayed because the image path is not good. The problem comes from the file MIMEMapping.java. When the load is done at the beginning, if the mime mapping node doesn't contains an "icon" nodedata it takes by default general.png instead of the constant DEFAULT_ICON that contains the full path. See the patch in attachment. |
| Comments |
| Comment by Jeff Snider [ 21/Feb/08 ] |
|
We ran into a similar issue with the download paragraph caused by this same thing. We took a slightly different approach by removing the default value entirely from here and let the getMIMETypeIcon() function below handle the situation, since it was already written to do so. Same outcome either way. We've tested this in our dev systems and it takes care of both the dms and the download paragraph. |
| Comment by Magnolia International [ 10/Sep/08 ] |
|
Fix will be available with Magnolia 3.5.9 and 3.6.2 - thanks for the patches ! (applied to trunk and 3.5 branch) |