[ZBX-6312] potentially incorrect note about imagerotate Created: 2013 Feb 23  Updated: 2024 Apr 10  Resolved: 2017 Oct 31

Status: Closed
Project: ZABBIX BUGS AND ISSUES
Component/s: Documentation (D)
Affects Version/s: None
Fix Version/s: 4.0 (plan)

Type: Documentation task Priority: Minor
Reporter: richlv Assignee: Natalja Cernohajeva (Inactive)
Resolution: Fixed Votes: 0
Labels: font, php, php-gd
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Team: Team A
Sprint: Sprint 19
Story Points: 0.1

 Description   

https://www.zabbix.com/documentation/2.2/manual/installation/requirements says :
"For other fonts than the default DejaVu, PHP function imagerotate might be required. If it is missing, these fonts might be rendered incorrectly in Monitoring ? Overview header and other locations."

this was changed at some point.
a) it should be figured out which version changed this, and note should reflect this;
b) all locations where this php function could be used should be listed;
c) documentation for all relevant versions should be updated

 Comments   
Comment by Oleksii Zagorskyi [ 2013 Feb 24 ]

Also note my reported issue https://bugs.php.net/bug.php?id=62167

I'd suggest to remove from zabbix doc a sentence that bundled version of the GD library is required.

Comment by Oleksii Zagorskyi [ 2014 Feb 01 ]

Note that the PHP documentation indeed had a mistake and issue I reported above currently is fixed.
So second sentence form the note Rich mentioned could be removed for 2.2 documentation for sure.

Comment by Pavels Jelisejevs (Inactive) [ 2014 Mar 20 ]

The note is indeed incorrect: we don't use imagerotate() for rendering vertical texts anymore. But we do use it for graphs, and now, if the functions is not available, a fatal error will be triggered.

I've stumbled upon this issue while working on ZBX-7896. I couldn't figure out which PHP or GD packages support the imagerotate() function. Currently, I think that we shouldn't rely on the imagerotate() function of the GD library and write our own replacement.

Comment by Pavels Jelisejevs (Inactive) [ 2014 Mar 20 ]

This also seems to be the reason why we have two constants for defining fonts: ZBX_GRAPH_FONT_NAME and ZBX_FONT_NAME. When we resolve the imagerotate() function problem, we could also unify these constants.

Comment by Natalja Cernohajeva (Inactive) [ 2017 Oct 27 ]

natalja.cernohajeva : imagerotate() function is used only for graphs since Zabbix 2.0.4 version. Documentation update has been made for all supported Zabbix versions:

RESOLVED
(Please reopen if required)

sasha CLOSED

Generated at Wed Apr 24 05:55:46 EEST 2024 using Jira 9.12.4#9120004-sha1:625303b708afdb767e17cb2838290c41888e9ff0.