Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[TASK] Use IconFactory for TYPO3 > 6 #26

Merged
merged 3 commits into from
Oct 10, 2016
Merged

[TASK] Use IconFactory for TYPO3 > 6 #26

merged 3 commits into from
Oct 10, 2016

Conversation

flossels
Copy link
Contributor

No description provided.

Copy link
Owner

@helhum helhum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than the missing hsc it looks good. Will test now.

$imgTag = '<img' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($backPath, $icon, $iconSize);

if ($title !== '') {
$imgTag .= 'title="' . $title . '" ';
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please apply htmlspechialchars to title

}

if ($alt !== '') {
$imgTag .= 'alt="' . $alt . '" ';
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please apply htmlspechialchars to alt

Copy link
Owner

@helhum helhum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Works now for me

@helhum helhum merged commit 0f876cf into helhum:master Oct 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants