Commit Graph

436 Commits (61fa1f5dc893d63674c3302978719d094e221223)

Author SHA1 Message Date
Gerhard Killesreiter 784202418f #50771, Wrong user name in admin/comment, patach by Markus Petrux 2006-02-27 14:54:24 +00:00
Dries Buytaert 66576708a0 - Patch #51347 by eafarris: fixed typo: coment -> comment. 2006-02-26 19:52:09 +00:00
Dries Buytaert 3489651659 - Patch #46690 by Zen: 'user comments' -> 'comment settings'. 2006-02-26 19:47:55 +00:00
Dries Buytaert c94a2dabb0 - Patch #50755 by markus: = -> ==. 2006-02-22 19:02:58 +00:00
Dries Buytaert cb05945caf - Patch #47699 by wtanaka: performance improvement. 2006-02-22 10:05:06 +00:00
Dries Buytaert 3603fe1bd8 - Patch #49912: www.drupal.org -> drupal.org. (Today's critical bugfix #5.) 2006-02-21 18:46:54 +00:00
Dries Buytaert 7489c87476 - Patch #43572 by Tobias: don't accept spaces. 2006-02-14 19:17:00 +00:00
Dries Buytaert 583d5a4ee9 - Patch #48622 by adrian: remove drupal_goto from _submit functions. 2006-02-10 05:25:57 +00:00
Steven Wittens 1819de963f - #48518: Editing own comment broken under postgresl 2006-02-09 09:09:03 +00:00
Steven Wittens c9a19e3084 - #48239: Comment thread coding inefficient 2006-02-09 08:33:36 +00:00
Dries Buytaert 91e152b8b8 - Patch #47705 by DriesK: comment moderation/approval wasn't working due to a bug with its forms. 2006-02-04 09:54:16 +00:00
Dries Buytaert baa2ecc81c - Patch #47045 by Cvbge: fixed problem with deleting comments. 2006-02-01 14:04:50 +00:00
Dries Buytaert a6dac9d2ab - Modified patch #46227 by chx: comment token is broken. 2006-01-31 15:26:39 +00:00
Dries Buytaert 20154df528 - Modified patch #45723 by wtanaka: creating a comment, editing a comment, or anything that validates a comment fails on PHP5. 2006-01-26 08:48:57 +00:00
Dries Buytaert 5d43619ca7 - Patch #40752 by moshe/wtanaka: login and regsiter links on each comment don't redirect back to the topic. 2006-01-26 08:27:58 +00:00
Dries Buytaert b8832d76a0 - Patch #40753 by wtanaka/killes: #comment-form has disappeared from comment.module 2006-01-25 08:54:40 +00:00
Dries Buytaert b6b98bb60d - Patch #44101 by Cvbge: deleting a comment creates incorrect entry in node_comment_statistics. 2006-01-22 07:26:33 +00:00
Dries Buytaert 4da1540a9f - Patch #45380 by Neil: usability improvement: disable anonymous commenting options when anonymous people can't comment. 2006-01-20 09:33:03 +00:00
Dries Buytaert 8c02d4ec93 - Patch #45530 by Morbus: filter_form shouldn't default to #weight 0
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.
2006-01-20 09:04:34 +00:00
Dries Buytaert e84a98a22b - Patch #45349 by Morbus Iff: input filters aren't sorting correctly infForms API. 2006-01-19 08:54:41 +00:00
Dries Buytaert 47dd0142e6 - Patch #43325 by chx/drumm/merlinofchaos: critical bugfix: made editing comments work. 2006-01-18 15:09:39 +00:00
Dries Buytaert cb45cf58fb - Patch #40755 by wtanaka: make the comment block themable. 2006-01-17 18:02:23 +00:00
Dries Buytaert 226a26a556 - Removed reference to comment moderation. 2006-01-10 19:41:15 +00:00
Steven Wittens 49e5e1cd47 - #43097: restore admin/comment tablesort 2006-01-08 00:29:37 +00:00
Steven Wittens 030fd1798a - Comment bug: anchor links should be specified using $fragment in url() and drupal_goto(). 2006-01-06 01:14:17 +00:00
Dries Buytaert 24899dba42 - Patch #40563 by chx, merlingofchaos, et al: bug fix: fixed the default comment settings on node sumbission forms. 2006-01-05 14:35:55 +00:00
Dries Buytaert 8d7ff5aea2 - Patch #39743 by merlinofchaos: critical bugfix: fixed comment ordering setting. 2006-01-05 10:53:47 +00:00
Dries Buytaert 72ed68d48b - Patch #42971 by markus_petrux: input format not correctly shown when editing a comment. 2006-01-03 23:04:23 +00:00
Dries Buytaert ea2d791095 - Patch 41169 by Chris: got rid of expensvie object 2 array casts. 2005-12-31 10:48:56 +00:00
Dries Buytaert 6164074655 - Patch #38999 by saerdna: There's no need that users with the "administer comments" role has to go via admin/comments to delete comments. this patch simply adds the delete link on each comment if you have administer comments role. "access administration pages" not needed; because if they not have it they can still delete from admin/comments. 2005-12-19 10:12:52 +00:00
Dries Buytaert 21a9442bbb - Patch #41081 by billturner: fixed problem with the comment settings form. 2005-12-15 16:31:01 +00:00
Dries Buytaert 922a46d281 - Patch #34920 by Morbus/webchick/tangent: fixed order of form fields. 2005-12-15 16:24:40 +00:00
Dries Buytaert 5accfa08f5 - Patch #41133 by DriesK: as of PHP 5.1.0, strtotime() returns FALSE on failure instead of -1.
Changed code to be compatible with all versions.
2005-12-15 16:02:50 +00:00
Dries Buytaert 764f1177ef - Patch #40631 by Chris Johnson: is_array() slower than isset() or empty(). 2005-12-14 20:10:45 +00:00
Dries Buytaert 3ae534a64e - Patch #28595 by Cvbge: fixed problems with bulk comment operations. 2005-12-08 09:11:32 +00:00
Dries Buytaert e019ce5107 - Patch #28595 by Jeremy: added support for bulk operations on comments. 2005-12-07 05:35:02 +00:00
Dries Buytaert 28fdb3975e - Patch #39778 by chx: obliterate nodeapi op form in favor of the forms API's way of doing things. Tested with help from webchick. 2005-12-05 09:11:33 +00:00
Dries Buytaert 80e02e6e73 - Patch #36434 by assimonds: fixed problem with editing signatures. 2005-12-03 19:39:15 +00:00
Dries Buytaert 26bd449860 - Patch #39576 by chx: rename '_execute' to '_submit' and '#execute' to '#submit'. 2005-12-02 15:21:01 +00:00
Dries Buytaert 8784a7d13a - Patch #39024 by Zen: comments that should be in the moderation queue get published. 2005-11-27 11:45:03 +00:00
Dries Buytaert c647b39c2c - Patch #37798 by Neil: fixed problem with node settings. 2005-11-27 11:11:46 +00:00
Dries Buytaert 247e012c35 - Patch #39013 by drumm: more logical naming of themeable functions 2005-11-27 11:07:02 +00:00
Dries Buytaert b1b1e2045f - Patch #37798 by Neil: allow non-workflow form elements on the node type settings page. 2005-11-25 10:11:59 +00:00
Dries Buytaert bd2d91807e - Patch #38575 by Neil: fixed infinite loop in comment.module. 2005-11-23 08:23:45 +00:00
Dries Buytaert 62e5c58777 - Patch #38491 by Neil: refactor comment viewing controls to _execute model. 2005-11-22 15:48:46 +00:00
Dries Buytaert 9f8fcab8db - Patch #35726 by asimmonds: post_process -> build_after 2005-11-22 15:13:15 +00:00
Dries Buytaert 28f973a986 - Patch #35726 by asimmonds/chx: converted the comment module to the forms API's _execute model. 2005-11-21 08:32:31 +00:00
Dries Buytaert e900d28640 - Patch #34295 by assimonds: added missing defines to comment.module. Improves
readability of the code.
2005-11-15 07:29:47 +00:00
Dries Buytaert d5d0149abe - Patch #34295 by DrupalJapan and chx:
+ Fixed problem with configuration of comment display options.
   + Introduced defines for the various comment settings.
2005-11-13 09:05:38 +00:00
Steven Wittens 21bed11e66 - Add missing url()s around form actions (these would break without clean URLs) 2005-11-13 02:32:18 +00:00