Comments

RavindraSingh created an issue. See original summary.

RavindraSingh’s picture

flexslider_picture.module

No Problems Found
HIDESITES/ALL/MODULES/CONTRIB/PICTURE/FLEXSLIDER_PICTURE/THEME/FLEXSLIDER_PICTURE.THEME.INC
flexslider_picture.theme.inc

severity: criticalLine 27: Potential problem: trigger_error() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)
      trigger_error("Unable to load picture mapping {$optionset->mapping}.", E_USER_ERROR);
severity: criticalLine 44: Potential problem: trigger_error() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)
        trigger_error("Unable to load picture mapping {$optionset->options['colorboxImageStyle']}.", E_USER_ERROR);
SHOWSITES/ALL/MODULES/CONTRIB/PICTURE/FLEXSLIDER_PICTURE/FLEXSLIDER_PICTURE.INSTALL
HIDESITES/ALL/MODULES/CONTRIB/PICTURE/PICTURE.MODULE
picture.module

severity: criticalLine 777: Potential problem: trigger_error() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)
      trigger_error("Unable to load picture mapping $mapping_name.", E_USER_ERROR);
severity: criticalLine 801: Potential problem: trigger_error() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)
      trigger_error("Unable to load picture mapping {$display['settings']['colorbox_settings']['colorbox_group']}.", E_USER_ERROR);
SHOWSITES/ALL/MODULES/CONTRIB/PICTURE/PICTUREFILL2/PICTUREFILL.JS
SHOWSITES/ALL/MODULES/CONTRIB/PICTURE/PICTUREFILL2/PICTUREFILL.MIN.JS
SHOWSITES/ALL/MODULES/CONTRIB/PICTURE/INCLUDES/PICTUREMAPPING.PHP
HIDESITES/ALL/MODULES/CONTRIB/PICTURE/FLEXSLIDER_PICTURE/THEME/FLEXSLIDER_PICTURE.THEME.INC
flexslider_picture.theme.inc

severity: criticalLine 27: Potential problem: trigger_error() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)
      trigger_error("Unable to load picture mapping {$optionset->mapping}.", E_USER_ERROR);
severity: criticalLine 44: Potential problem: trigger_error() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)
        trigger_error("Unable to load picture mapping {$optionset->options['colorboxImageStyle']}.", E_USER_ERROR);
SHOWSITES/ALL/MODULES/CONTRIB/PICTURE/FLEXSLIDER_PICTURE/FLEXSLIDER_PICTURE.INSTALL
HIDESITES/ALL/MODULES/CONTRIB/PICTURE/CTOOLS/PLUGINS/EXPORT_UI/PICTURE_MAPPING.INC
picture_mapping.inc

severity: normalLine -1: @file block missing (Drupal Docs)
SHOWSITES/ALL/MODULES/CONTRIB/PICTURE/CKEDITOR/PLUGINS/PLUGIN.JS
SHOWSITES/ALL/MODULES/CONTRIB/PICTURE/PICTURE.ADMIN.INC
SHOWSITES/ALL/MODULES/CONTRIB/PICTURE/PICTURE.DRUSH.INC
HIDESITES/ALL/MODULES/CONTRIB/PICTURE/PICTURE.FILE_ENTITY_1.INC
picture.file_entity_1.inc

severity: criticalLine 50: Potential problem: trigger_error() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)
      trigger_error("Unable to load picture mapping $group_name.", E_USER_ERROR);
RavindraSingh’s picture

Fixed in this patch.

Screenshot is attached

Coding Standards

RavindraSingh’s picture

Status: Active » Needs review
attiks’s picture

Status: Needs review » Fixed

Thanks

  • attiks committed 4f91f87 on 7.x-2.x authored by RavindraSingh
    Issue #2556981 by RavindraSingh: few potential issues which needs to be...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.