? module_builder-codestyle-fix.patch
Index: module_builder.module
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/module_builder/module_builder.module,v
retrieving revision 1.41.2.14
diff -u -p -r1.41.2.14 module_builder.module
--- module_builder.module	20 Jul 2009 21:09:19 -0000	1.41.2.14
+++ module_builder.module	26 Jul 2009 08:35:33 -0000
@@ -315,7 +315,7 @@ function module_builder_page_input($form
  }
 
   // Include CSS for formatting
-  drupal_add_css(drupal_get_path('module', 'module_builder') . '/includes/module_builder.css');
+  drupal_add_css(drupal_get_path('module', 'module_builder') .'/includes/module_builder.css');
 
   // Mark form as fresh to enable JS clearing of fields with sample text.
   if ($form_state['clicked_button']['#name'] != 'back') {
@@ -381,11 +381,11 @@ function module_builder_page_input($form
     '#description' => t('Selecting one or more of these features will automatically select appropriate hooks for you.'),
   );
 
-  drupal_add_js($path . '/includes/module_builder.js');
+  drupal_add_js($path .'/includes/module_builder.js');
 
   // Get list of hook presets from installed template.
   $hook_presets = module_builder_parse_template($template_file);
-  foreach ($hook_presets as $hook_preset_name => $hook_preset ){
+  foreach ($hook_presets as $hook_preset_name => $hook_preset) {
     $hooks = explode("\n", $hook_preset['template']);
     $hook_array = array();
     foreach ($hooks as $hook) {
@@ -659,7 +659,7 @@ function Xmodule_builder_page_input($for
   _module_builder_check_settings();
 
   // Include CSS for formatting
-  drupal_add_css(drupal_get_path('module', 'module_builder') . '/includes/module_builder.css');
+  drupal_add_css(drupal_get_path('module', 'module_builder') .'/includes/module_builder.css');
 
   // Module properties
   $form['module_root_name'] = array(
@@ -714,11 +714,11 @@ function Xmodule_builder_page_input($for
     '#description' => t('Selecting one or more of these features will automatically select appropriate hooks for you.'),
   );
 
-  drupal_add_js($path . '/includes/module_builder.js');
+  drupal_add_js($path .'/includes/module_builder.js');
 
   // Get list of hook groups from installed template.
   $hook_groups = module_builder_parse_template($template_file);
-  foreach ($hook_groups as $hook_group_name => $hook_group ){
+  foreach ($hook_groups as $hook_group_name => $hook_group) {
     $hooks = explode("\n", $hook_group['template']);
     $hook_array = array();
     foreach ($hooks as $hook) {
@@ -895,7 +895,7 @@ function Xmodule_builder_page_download($
     $file_content = $form_values['module_code'];
     $file_ext = '.module';
   }
-  else if ($form_values['op'] == t('Download info file')) {
+  elseif ($form_values['op'] == t('Download info file')) {
     $file_content = $form_values['module_info'];
     $file_ext = '.info';
   }
@@ -974,12 +974,12 @@ function _module_builder_write_file($bas
  */
 function Xmodule_builder_page_write($form, $form_values) {
   ####### bug!!!!!!!!!
-  dpr('writing page: ' . $form_values['op']);
+  dpr('writing page: '. $form_values['op']);
 
   if ($form_values['op'] == t('Write module file')) {
     _module_builder_write_file($form_values['module_root_name'], '.module', $form_values['module_code']);
   }
-  else if ($form_values['op'] == t('Write info file')) {
+  elseif ($form_values['op'] == t('Write info file')) {
     _module_builder_write_file($form_values['module_root_name'], '.info', $form_values['module_info']);
   }
   else {
@@ -1024,11 +1024,11 @@ function Xmodule_builder_page_validate($
     }
 
     // Make sure at least one hook was chosen
-    $hook_selected = false;
+    $hook_selected = FALSE;
     foreach ($form_values['hooks'] as $file => $hooks) {
       foreach ($hooks as $hook) {
         if ($hook == 1) {
-          $hook_selected = true;
+          $hook_selected = TRUE;
           break;
         }
       }
@@ -1150,7 +1150,7 @@ function module_builder_generate_module(
 ";
 
     // function declaration: put in the module name, add closing brace, decode html entities
-    $declaration = str_replace('hook', $module_data['module_root_name'], $hook['declaration']) . ' {';
+    $declaration = str_replace('hook', $module_data['module_root_name'], $hook['declaration']) .' {';
     $code .= htmlspecialchars_decode($declaration);
 
     // See if function bodies exist; if so, use function bodies from template
@@ -1329,7 +1329,7 @@ function module_builder_get_doc_files($p
 
   if (is_dir($path)) {
     if ($dh = opendir($path)) {
-      while (($file = readdir($dh)) !== false) {
+      while (($file = readdir($dh)) !== FALSE) {
         // Ignore files that don't make sense to include
         if ($file != '.' && $file != '..' && $file != 'CVS' && $file != 'install.php') {
           $files[] = $file;
Index: drush/module_builder.drush.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/module_builder/drush/Attic/module_builder.drush.inc,v
retrieving revision 1.1.2.2
diff -u -p -r1.1.2.2 module_builder.drush.inc
--- drush/module_builder.drush.inc	22 Jul 2009 20:25:43 -0000	1.1.2.2
+++ drush/module_builder.drush.inc	26 Jul 2009 08:35:33 -0000
@@ -162,34 +162,34 @@ function module_builder_callback_info($c
  * Output generated text, to terminal or to file.
  */
 function module_builder_drush_output_code($module_root_name, $type, $code) {
-  
-  $filename = $module_root_name . '.' . $type;
-  
+
+  $filename = $module_root_name .'.'. $type;
+
   // Output to terminal
   if (!drush_get_option('quiet')) {
     drush_print("Proposed $filename:");
     drush_print_r($code);
   }
-  
+
   // Write to file
   if (drush_get_option('write')) {
     $module_dir = pm_dl_destination('module');
     $module_dir .= $module_root_name;
-    
+
     if (!is_dir($module_dir)) {
       @drush_op('mkdir', $module_dir, 0777);
       //drush_print("Module directory $module_dir created");
     }
 
-    $filepath = $module_dir . '/' . $filename;
+    $filepath = $module_dir .'/'. $filename;
 
     // if file exists, ask for whether to overwrite
     if (file_exists($filepath)) {
-      if (!drush_confirm(dt('File ' . $filename . ' exists. Do you really want to overwrite?'))) {
-        return; 
+      if (!drush_confirm(dt('File '. $filename .' exists. Do you really want to overwrite?'))) {
+        return;
       }
     }
-    
+
     file_put_contents($filepath, $code);
   }
 }
@@ -201,13 +201,13 @@ function module_builder_drush_output_cod
  * @return The entered string.
  */
 function drush_input($msg, $required = FALSE, $indent = 0) {
-  print str_repeat(' ', $indent) . (string)$msg . ": ";
+  print str_repeat(' ', $indent) . (string)$msg .': ';
 
   while ($line = trim(fgets(STDIN))) {
     if (!$required or strlen($line) > 0) {
       return $line;
     }
     print 'we never get here wtf?';
-    print str_repeat(' ', $indent) . (string)$msg . ": ";
+    print str_repeat(' ', $indent) . (string)$msg .': ';
   }
-}
\ No newline at end of file
+}
