The following piece of code does not follow the Coding Standards for indentation.

function user_register_form($form, &$form_state) {
  global $user;

  $admin = user_access('administer users');

  // Pass access information to the submit handler. Running an access check
  // inside the submit function interferes with form processing and breaks
  // hook_form_alter().
  $form['administer_users'] = array(
     '#type' => 'value',
     '#value' => $admin,
  );
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pvmchau’s picture

Status: Active » Needs review
FileSize
17.76 KB

Update coding standards for user.module

thehong’s picture

Status: Needs review » Needs work
+  
+  // libxml2 ver.2.7.0 - 2.7.2 stripping leading angle brackets bug patch
+  if (
+      LIBXML_DOTTED_VERSION == '2.7.0' ||
+      LIBXML_DOTTED_VERSION == '2.7.1' ||
+      LIBXML_DOTTED_VERSION == '2.7.2' ||
+      (LIBXML_DOTTED_VERSION == '2.7.3' && version_compare(PHP_VERSION, '5.2.9', '<' ))
+  ) {
+    $data = str_replace('&lt;', '&#60;', $data);
+    $data = str_replace('&gt;', '&#62;', $data);
+    $data = str_replace('&amp;', '&#38;', $data);
+  }

I think you should remove those lines.

eltermann’s picture

Status: Needs work » Needs review
FileSize
554 bytes
nkhanh90’s picture

parthipanramesh’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Latest patch does apply indentions correctly.

David_Rothstein’s picture

Status: Reviewed & tested by the community » Needs work

Nope, oops... it needs to be two spaces of indentation, not one space :)

lachezar.valchev’s picture

Assigned: Unassigned » lachezar.valchev
gaurav.goyal’s picture

gaurav.goyal’s picture

Status: Needs work » Needs review
FileSize
556 bytes

Indentation Corrected.

David_Rothstein’s picture

Status: Needs review » Reviewed & tested by the community

That looks great - thanks!

The bug doesn't appear to be present in Drupal 8 (must have been fixed somewhere along the way).

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 9: 1988456_user_register_form_coding_standard_fix.patch, failed testing.

David_Rothstein’s picture

Status: Needs work » Reviewed & tested by the community
gaurav.goyal’s picture

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 9: 1988456_user_register_form_coding_standard_fix.patch, failed testing.

David_Rothstein’s picture

Status: Needs work » Reviewed & tested by the community

Likely testbot glitch - moving back to RTBC.

David_Rothstein’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x - thanks!

  • David_Rothstein committed d1a4193 on 7.x
    Issue #1988456 by gaurav.goyal, eltermann, pvmchau: Non-standard...

Status: Fixed » Closed (fixed)

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