Bug 44070 – Portal does not show .png files as logos for entries

Bug 44070 - Portal does not show .png files as logos for entries


Summary:	Portal does not show .png files as logos for entries

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	Portal
Version:	UCS 4.2
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS 4.2
Assigned To:	Alexander Kläser
QA Contact:	Stefan Gohmann

URL:
Keywords:	interim-4

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2017-03-24 14:23 CET by Dirk Wiesenthal
Modified:	2017-04-04 18:28 CEST (History)
CC List:	1 user (show)

See Also:
What kind of report is it?:	Bug Report
What type of bug is this?:	1: Cosmetic issue or missing function but workaround exists
Who will be affected by this bug?:	2: Will only affect a few installed domains
How will those affected feel about the bug?:	2: A Pain – users won’t like this once they notice it
User Pain:	0.023
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:	Yes
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:	2017032921000611
Bug group (optional):
Max CVSS v3 score:

Attachments
Placeholder images when PNG images should be used (318.49 KB, image/png) 2017-03-24 14:23 CET, Dirk Wiesenthal	Details
Screenshot with different image types working :) (893.26 KB, image/png) 2017-03-31 01:51 CEST, Alexander Kläser	Details
Detect GIF magic number (1.64 KB, patch) 2017-03-31 01:52 CEST, Alexander Kläser	Details \| Diff
Fix representation of other image types than SVG (2.01 KB, patch) 2017-03-31 01:52 CEST, Alexander Kläser	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Dirk Wiesenthal

2017-03-24 14:23:58 CET

Created attachment 8639 [details]
Placeholder images when PNG images should be used

App audriga:

ucr set ucs/web/overview/entries/service/audriga-groupware-migration/icon='/icon/50x50/groupware-migration-icon.png'

This (together with a group of other UCRVs like /link) creates a settings/portal_entry. The logo is saved as
  /usr/share/univention-portal/icons/entries/audriga-groupware-migration.png

The source file is
  /var/www/icon/50x50/groupware-migration-icon.png

Both files exist. Yet, the portal refuses to show it. It only shows SVG images. Otherwise a placeholder is shown.

Comment 1 Dirk Wiesenthal

2017-03-30 03:15:17 CEST

Brought up by an App Provider

Comment 2 Alexander Kläser

2017-03-31 01:51:10 CEST

Created attachment 8718 [details]
Screenshot with different image types working :)

Its working... see the commit... I just have trouble connecting via VPN ATM.

Comment 3 Alexander Kläser

2017-03-31 01:52:07 CEST

Created attachment 8719 [details]
Detect GIF magic number

Comment 4 Alexander Kläser

2017-03-31 01:52:35 CEST

Created attachment 8720 [details]
Fix representation of other image types than SVG

Comment 5 Alexander Kläser

2017-03-31 09:06:35 CEST

Fixed. PNG, JPEG, GIF and SVG images can now be uploaded. The Image widget recongizes these formats via their magic number.


univention-portal (1.0.11-3):
r78525 | Bug #44070: Fix representation of other image types than SVG

univention-web (1.0.42-7):
r78524 | Bug #44070: Recognize GIF as image type

changelog-4.2-0.xml:
r78526 | Bug #44070: Add bug number to changelog entry

Comment 6 Stefan Gohmann

2017-03-31 16:03:00 CEST

Excellent. I had a caching problem but now it works.

Comment 7 Stefan Gohmann

2017-04-04 18:28:27 CEST

UCS 4.2 has been released:
 https://docs.software-univention.de/release-notes-4.2-0-en.html
 https://docs.software-univention.de/release-notes-4.2-0-de.html

If this error occurs again, please use "Clone This Bug".