Loading...
 
Skip to main content

Icons are not being assigned properly due to recent commits

Status
Closed
Subject
Icons are not being assigned properly due to recent commits
Version
14.x Regression
Category
  • Regression
Feature
WYSIWYG (What You See is What You Get)
Resolution status
Fixed or Solved
Submitted by
taylordustin
Volunteered to solve
Jonny Bradley
Lastmod by
lindon
Rating
(0)
Description

I noticed after pulling the latest code from SourceForge that certain icons in the WYSIWYG and Wiki editors were not being set properly. A condition inside an if statement in /lib/toolbars/toolbarslib.php on the line 1709 was setting several icons to the plugin.php icon.

I began looking for the cause of the issue and discovered a series of several commits that had caused the bug. The icons in question are the features 'Image', 'File', 'Mouseover', 'Insert Module', 'HTML', 'Include', 'Code', and 'Remarks Box'.

The problem code for each of the above features can be found in the lib/wiki-plugins/ directory and is attributed to a single line of code in each file. The structure of the hash tables located in these files appears to have been changed from using

'icon' => 'path/to/icon'
to
'iconname' => 'iconset_definition'

Unfortunately, this has caused the regression on my site. To fix this, I can simply comment out the line with 'iconname', and replace it with the previous 'icon' line, however this increases the number of customizations on my site I would need to keep track of, and would prefer not to go down that route.

The revision numbers and files for the features in question are as follows:

'Image' = r56275 — /wiki-plugins/wikiplugin_img.php
'File' = r56269 — /wiki-plugins/wikiplugin_file.php
'Mouseover' = r56280 — /wiki-plugins/wikiplugin_mouseover.php
'Insert Module' = r56280 — /wiki-plugins/wikiplugin_module.php
'HTML' = r56273 — /wiki-plugins/wikiplugin_html.php
'Include' = r56276 — /wiki-plugins/wikiplugin_include.php
'Code' = r56254 — /wiki-plugins/wikiplugin_code.php
'Remarks Box' = r56291 — /wiki-plugins/wikiplugin_remarksbox.php

I looked into the new iconset definitions as well, and didn't notice any issues with it, so this is why I'm am submitting the bug report. Let me know if there is anything I can help with!

Solution

Hopefully r56351 will do it for the normal wiki toolbars, still trying to work out a fix for WYSIWYG...
Thanks for the excellent bug report!

Importance
6
Easy to solve?
6
Priority
36
Demonstrate Bug on Tiki 19+
Demonstrate Bug (older Tiki versions)
Ticket ID
5719
Created
Monday 28 September, 2015 19:37:14 UTC
by taylordustin
LastModif
Saturday 07 November, 2015 18:10:56 UTC


Show PHP error messages