Skip to content
This repository has been archived by the owner on Mar 17, 2018. It is now read-only.

Commit

Permalink
Merge pull request #2 from robsonsobral/patch-1
Browse files Browse the repository at this point in the history
Fix display of optiongroup in settings of P&T Dropdown.
  • Loading branch information
Brandon Kelly committed Nov 23, 2011
2 parents d7edea6 + 113e3df commit 5e5bde4
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions ee2/third_party/pt_field_pack/pt_fieldtype.php
Expand Up @@ -43,9 +43,8 @@ function options_setting($options=array(), $indent = '')
{
if ($r !== '') $r .= "\n";

// force strings
// force string
$name = (string) $name;
$label = (string) $label;

// is this just a blank option?
if ($name === '' && $label === '') $name = $label = ' ';
Expand All @@ -57,7 +56,7 @@ function options_setting($options=array(), $indent = '')
{
$r .= "\n".$this->options_setting($label, $indent.' ');
}
else if ($name !== $label)
else if ($name !== (string) $label)
{
$r .= ' : '.$label;
}
Expand Down

0 comments on commit 5e5bde4

Please sign in to comment.