by jcfiala:Various tweaks to link module to pass coder inspection.

link.module

@@ -64,7 +64,7 @@ function link_field_settings($op, $field) {
       $form['validate_url'] = array(
         '#type' => 'checkbox',
         '#title' => t('Validate URL'),
-        '#default_value' => isset($field['validate_url']) && ($field['validate_url'] !== '') ? $field['validate_url'] : true,
+        '#default_value' => isset($field['validate_url']) && ($field['validate_url'] !== '') ? $field['validate_url'] : TRUE,
         '#description' => t('If checked, the URL field will be verified as a valid URL during validation.'),
       );
 

tests/link.attribute.test

@@ -52,13 +52,13 @@ class LinkAttributeCrudTest extends ContentCrudTestCase {
    */
   private function outputScreenContents($description, $basename) {
     // This is a hack to get a directory that won't be cleaned up by simpletest
-    $file_dir = file_directory_path().'/../simpletest_output_pages';
+    $file_dir = file_directory_path() .'/../simpletest_output_pages';
     if (!is_dir($file_dir)) {
       mkdir($file_dir, 0777, TRUE);
     }
     $output_path = "$file_dir/$basename." . $this->randomName(10) . '.html';
     $rv = file_put_contents($output_path, $this->drupalGetContent());
-    $this->pass("$description: Contents of result page are ".l('here',$output_path));
+    $this->pass("$description: Contents of result page are ". l('here', $output_path));
   }
   
   private function assertLinkOnNode($field_name, $link_value, $message = '', $group = 'Other') {    

tests/link.crud_browser.test

 <?php
-//$Id$
+// $Id$
+
+/**
+ * @file
+ * Testing CRUD API in the browser.
+ */
 
 /**
  * Testing that users can not input bad URLs or labels

tests/link.token.test

 <?php
-//$Id$
+// $Id$
+
+/**
+ * @file
+ * Contains simpletests making sure token integration works.
+ */
 
 /**
  * Testing that tokens can be used in link titles