Commit dfe7dfd9 authored by Jonathan Smith's avatar Jonathan Smith
Browse files

Issue #3221220 by jonathan1055: Fix coding standards for Coder 8.3.13

parent ab1944f3

drupalci.yml

+1 −0
Changes for drupalci.yml: 1 added line, 0 removed lines.
Original line number Diff line number Diff line
@@ -5,6 +5,7 @@ build: 
      container_composer:

      phplint:

      phpcs:

        sniff-all-files: false

      eslint:

      csslint:

    testing:

phpcs.xml.dist

+7 −0
Changes for phpcs.xml.dist: 7 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -25,4 +25,11 @@ 
  <exclude-pattern>\/_+ignore</exclude-pattern>

  <exclude-pattern>interdif</exclude-pattern>



  <!-- Increase the allowed line length for inline array declarations. -->

  <rule ref="Drupal.Arrays.Array">

    <properties>

      <property name="lineLimit" value="120"/>

    </properties>

  </rule>



</ruleset>

plugins/content_types/scheduler_form_pane.inc

+1 −0
Changes for plugins/content_types/scheduler_form_pane.inc: 1 added line, 0 removed lines.
Original line number Diff line number Diff line
@@ -7,6 +7,7 @@ 
 * This content type contains the publish and unpublish scheduling date fields.

 */



// phpcs:ignore DrupalPractice.CodeAnalysis.VariableAnalysis

$plugin = array(

  'single' => TRUE,

  'edit form' => 'scheduler_form_pane_node_form_menu_content_type_edit_form',

scheduler.admin.inc

+7 −2
Changes for scheduler.admin.inc: 7 added lines, 2 removed lines.
Original line number Diff line number Diff line
@@ -195,10 +195,15 @@ function scheduler_admin_submit($form, &$form_state) { 
  variable_set('scheduler_date_only_format', $date_only_format);



  if (empty($time_only_format)) {

    drupal_set_message(t('The date part of the Scheduler format is %date_part. There is no time part', array('%date_part' => $date_only_format)));

    drupal_set_message(t('The date part of the Scheduler format is %date_part. There is no time part', array(

      '%date_part' => $date_only_format,

    )));

  }

  else {

    drupal_set_message(t('The date part of the Scheduler format is %date_part and the time part is %time_part.', array('%date_part' => $date_only_format, '%time_part' => $time_only_format)));

    drupal_set_message(t('The date part of the Scheduler format is %date_part and the time part is %time_part.', array(

      '%date_part' => $date_only_format,

      '%time_part' => $time_only_format,

    )));

  }

}

scheduler.module

+1 −7
Changes for scheduler.module: 1 added line, 7 removed lines.
Original line number Diff line number Diff line
@@ -876,14 +876,8 @@ function scheduler_feeds_set_target($source, $entity, $target, $value, $mapping) 
  // then strtotime() must not be used, otherwise it returns the current time.

  if (!empty($value) && !is_numeric($value)) {

    if (!$timestamp = strtotime($value)) {

      throw new FeedsValidationException(

      // Throw an exception if the date format was not recognized.

        t('Value %value for @source could not be converted to a valid %target date.', array(

          '@source' => $mapping['source'],

          '%value' => $value,

          '%target' => $target,

        )

      ));

      throw new FeedsValidationException("Value $value for {$mapping['source']} could not be converted to a valid Scheduler $target date.");

    }

  }

  else {