Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I found that In string length access plugin "Equal to" option not work
In ctools_string_length_ctools_access_settings operator defenition following
$form['settings']['operator'] = array(
'#type' => 'radios',
'#title' => t('Operator'),
'#options' => array(
......
'=' => t('Equal to'),
......
),
'#default_value' => $conf['operator'],
);
but in ctools_string_length_ctools_access_check operator switch following
switch ($conf['operator']) {
.................
case '==':
return $length == $conf['length'];
..............
}
Comments
Comment #1
niko- CreditAttribution: niko- commentedPatches for ctools drupal 7/8.1/8.2 versions attached
Not sure if any test need for this fixes
Comment #2
andypostYeah! Serious typo.
is there any tests?
Comment #3
japerryhmm seeing that the check should -always- fail because the operator didn't match '==' I think this is good. I added a return FALSE at the end to catch any errant settings, and to be less confusing to developers. if this looks good I'll push that up.
Comment #4
niko- CreditAttribution: niko- commentedHi Jakob
Looks good.
Comment #5
mrjmd CreditAttribution: mrjmd commentedComment #6
japerryAwesome. Fixed.