Skip to content
  1. Feb 25, 2011
  2. Mar 25, 2007
  3. Nov 14, 2006
  4. Nov 12, 2006
  5. Nov 01, 2006
  6. Oct 18, 2006
  7. Aug 20, 2006
  8. Jul 02, 2006
  9. May 21, 2006
  10. Apr 26, 2006
  11. Apr 17, 2006
  12. Apr 04, 2006
  13. Mar 04, 2006
    • Dries Buytaert's avatar
      - Patch #52381 by Zen: · a9ff6cd3
      Dries Buytaert authored
          * Converts the block administration page to the fapi model.
          * Removes some 'type=markup' elements.
          * Adds a form_render($form['form_id']); in the theme function. Adding a form_render($form) just outputs an unthemed form. I suspect this is because of the heavy nesting of form elements and the rather dodgy array declaration, but I'm unsure.
          * Documentation/language fixes.
      a9ff6cd3
  14. Feb 21, 2006
  15. Feb 10, 2006
  16. Jan 29, 2006
  17. Jan 26, 2006
  18. Jan 24, 2006
  19. Jan 20, 2006
    • Dries Buytaert's avatar
      - Patch #45530 by Morbus: filter_form shouldn't default to #weight 0 · 8c02d4ec
      Dries Buytaert authored
      When a form element doesn't specify a #weight, it is assumed internally as #weight 0. However, to ensure that our form elements display visually *as they were defined in the array* we, in form_builder, count the number of elements, divide by 1000, and set that as the weight:
      
      # Assign a decimal placeholder weight to preserve original array order
      if (!isset($form[$key]['#weight'])) {
        $form[$key]['#weight'] = $count/1000;
      }
      
      The above code will set the #weights of elements that have not defined a weight to something like 0 (first element in array definition), 0.001, 0.002, and so on. However, anytime a form element *explicitly* defines a #weight of 0, that #weight is kept at exactly 0, which would cause that form element to appear BEFORE the elements that didn't have a #weight defined (and thus received a #weight such as 0.002).
      
      Consider the following pseudo example:
      
      $form['game_title'] = array(
          '#type' => 'textfield',
          ...
          );
      $form['game_description'] = array(
          '#type' => 'textarea',
          ...
          );
      $form['game_format'] = filter_form(variable_get('game_format', NULL));
      return $form;
      
      Here, we're not definiing weights on our two textfields. We then add an filter_form. The second parameter of the filter_form is $weight, which defaults to 0. After this $form hits form_builder, we have weights 0 (game_title), 0.001 (game_description), and 0 (filter_form) respectively. This is then sorted by weight, which causes filter_form (the third element in the array) to appear BEFORE game_description (0 is lighter than 0.001).
      
      The short lesson is: explicitly defining #weight 0 for a form element is probably a bad idea. This patch changes the default #weight of filter_form to NULL, instead of 0, and also removes any other explicit setting of #weight to 0 in core.
      8c02d4ec
  20. Jan 19, 2006
  21. Jan 12, 2006
  22. Jan 08, 2006
  23. Jan 06, 2006
  24. Dec 14, 2005
  25. Dec 05, 2005
  26. Dec 02, 2005
  27. Nov 16, 2005
  28. Nov 13, 2005
  29. Nov 12, 2005
  30. Nov 03, 2005
  31. Nov 01, 2005
  32. Oct 22, 2005
  33. Oct 11, 2005
  34. Oct 07, 2005
  35. Sep 14, 2005
  36. Sep 12, 2005
  37. Aug 25, 2005
  38. Aug 16, 2005