summaryrefslogtreecommitdiff
path: root/modules/comment.module
Commit message (Collapse)AuthorAge
* - Modified patch #18215 by ankur: make sure the comment statistics update ↵Dries Buytaert2005-04-08
| | | | after approval in the moderation queue.
* - Patch by Drumm: change the wording for clarity.Dries Buytaert2005-04-08
|
* - By default, do not show comment controls.Dries Buytaert2005-04-07
|
* - Patch #20103 by Neil: Removing another place where we have hard coded HTML ↵Dries Buytaert2005-04-07
| | | | instead of the form functions. The unpublish selection actually didn't seem to work, so I removed it since we have delete. It should be implemented as a link anyway.
* - Patch #19451 by JonBob: improved consistency of module descriptions. We ↵Dries Buytaert2005-04-01
| | | | should write guidelines for this -- maybe in the PHPDoc code of the _help hook.
* - #18817: Clean up plain-text checking (see drupal-devel!)Steven Wittens2005-03-31
|
* - Patch #19009 by chx: fixed a bug with comment replies letting one access ↵Dries Buytaert2005-03-20
| | | | nodes.
* - #18939 (Stefan): Always use paragraph tags around page help text.Steven Wittens2005-03-18
|
* - Patch #17161 by Morbus: increase 40-character limit in URL field for comments.Dries Buytaert2005-03-16
|
* - Patch #14109 by wulff/Morbus: added missing t() funcitonDries Buytaert2005-03-16
|
* - Patch #18692 by Goba: old comments (comments for nodes having no entry in ↵Dries Buytaert2005-03-12
| | | | the history table anymore) are counted as new comments, since the timestamp retrieved from the history table is used to check for new comments (which is 0, if an entry was not found). This patch sets the timestamp used to check for new comments to the NODE_NEW_LIMIT value at least (which is when the history table was/should have been pruned).
* - Patch #18676 by frjo: When anonymuos users are allowed to add comments ↵Dries Buytaert2005-03-10
| | | | there are a required e-mail field. This field are, as far as i can see, not shown publicly. A user asked me about this and after confirming that it is not shown publicly I added a description that inform the users of this. I have included a patch that add the description "The content of this field is kept private and will not be shown publicly". This is the same description that is used by the profile module.
* #18329: Unify confirmation messages (and make them themable)Steven Wittens2005-03-03
|
* - #17877: Auto-fill comment subject during _validate rather than _save.Steven Wittens2005-02-27
|
* - Patch #17208 by asimmonds: help text fixes:Dries Buytaert2005-02-12
| | | | | | | | | - permissions menu link updates in a number of modules help - anchor link fix in distributed auth help - "my account" link fix in user help - spelling correction in tracker.module help - I also changed 'admin/access/perms' to 'admin/access/permissions'.
* - Patch #11366 by Junyor: fixed twin comment problem in HEAD.Dries Buytaert2005-02-11
|
* - Patch #16246 by Moshe: added mechanism to direct the user back to the ↵Dries Buytaert2005-02-01
| | | | | | referring page after completing a form. (Moshe: the patch against node.module failed to apply.)
* - Patch by Jose A Reyero: fixed probem with db_rewriste_sql() in the ↵Dries Buytaert2005-01-31
| | | | comment.module.
* - Patch #16253 by Goba: incremental improvements to the new content markers.Dries Buytaert2005-01-30
| | | | Goba: it would be nice if one of the core themes would showcase this functionality. ;-)
* - Patch #16111 by chx: generalized node_rewrite_query to db_rewrite_query.Dries Buytaert2005-01-29
|
* - Patch #16253 by Goba: this simple and straightforward patch adds the ↵Dries Buytaert2005-01-27
| | | | ability to define different types of markers (while retaining the old default behaviour of the new and required markers to look the same). Someone with enough time on his hands might be able to partition the new marker to a real new marker and a changed marker (since node_is_new() returns TRUE even if nodes changed, and not only when they are new). This is the base on which the new patch can be worked though.
* - Modified patch #14170 by Neil: improved default workflow (default node ↵Dries Buytaert2005-01-24
| | | | | | type) configuration page. Modified the patch to remove some redundant code, to translate strings, and to better use the menu system.
* - Fix tracker pager count query being wrong (it did not return a count but a ↵Steven Wittens2005-01-19
| | | | | | set, and it counted nodes with more than 1 comment as multiple nodes). - Clean up various SQL queries: removing literally inserted data (db_escape_string is evil!), fixing single "%" which should be "%%", fixing integers being compared as strings.
* - The real code fixes ;)Steven Wittens2005-01-17
|
* - Patch #14731 by chx: made it possible to rewrite node queries.Dries Buytaert2005-01-16
|
* - Fix comment pager counting unpublished comments.Steven Wittens2005-01-12
|
* - Fix comment pager counting unpublished comments.Steven Wittens2005-01-12
|
* - Updated two watchdog calls.Dries Buytaert2005-01-09
|
* - Patch #13260 by UnConeD: watchdog module improvements.Dries Buytaert2005-01-09
| | | | | | | | | | We added a 'severity' column to watchdog(): watchdog($type, $message, $link) --> watchdog($type, $message, $severity, $link); * Specify a severity in case you are reporting a warning or error. * The $link-parameter is now the fourth parameter instead of the third. TODO: document this in the upgrade guide.
* - Patch #14917 by UnConeD/Steven:Dries Buytaert2004-12-31
| | | | | | | | 1) The different types of search, which used to be radio button options in the search form, are now subtabs of "search" (default "search/node"). This seems better from a UI point of view, but also has another advantage: modules which implement a custom search form (flexinode, project) can add it as a subtab of search. This means that all search forms will be located in the same place, and also without needing an extra api call to search.module. 2) The current code was a bit hackish, as the indexing of comments along with nodes was hardcoded in node.module. Instead, I created a nodeapi operation "update index" which allows modules to add more data for a node that is being indexed. Comments are now indexed using this mechanism and from comment.module, which is a lot cleaner. 3) The search results format was also hardcoded to include "N comments". I replaced this with a nodeapi operation "search result" and moved the comment code to comment.module where it belongs. This op is quite useful, as for example I also modified upload.module to add "N attachments" to a search result if any are present.
* - Patch #14882 by Jeremy: removed some cruft: status = 2 is no more.Dries Buytaert2004-12-29
|
* - Patch #14710 by tangent: only show post button if preview is optional or ↵Dries Buytaert2004-12-23
| | | | if we are in preview mode. We show the post button in preview mode even if there are form errors so that optional form elements (e.g., captcha) can be updated in preview mode.
* Making comment moderation query PGSQL/ANSI compatible.Steven Wittens2004-12-20
|
* - Patch #13907 by Neil: less ways to set the page title.Dries Buytaert2004-12-15
| | | | | | | * Less logic in theme code. * Encourages use of the menu system. * Easier to find where a title or breadcrumb comes from in other people's code because there are less places to look. Look in menu and then grep for the appropriate set function. Looking for calls to theme_page() is hard because there are too many of them. * Very slightly more efficient.
* - Fixed typo: db_queryd() -> db_query()Dries Buytaert2004-12-11
|
* - Bugfix: make sure to update the node_comment_statistics table when a node ↵Dries Buytaert2004-12-11
| | | | is updated.
* - Refactored the queue module: removed the queue module's field from the ↵Dries Buytaert2004-12-07
| | | | | | | | | | node table. With help from Gerhard. - Slight addition to INSTALL.txt with regard to PHP versions. - Updated/reworded some node type descriptions as per Boris' suggestions. - Adding missing {} around a table name in update.php.
* - Patch #13816 by Neil: tidied up the comment module code a bit.Dries Buytaert2004-12-02
|
* - Patch #13647 by Goba:Dries Buytaert2004-11-29
| | | | | 1. Fixed broken watchdog calls: two watchdog calls omitted the type parameter, and thus injected logs into the type field, instead of the message field. 2. Removed t() functions from user contributed content.
* - Patch #12555 by svemir: fixed problem with the 'Preview comment' button ↵Dries Buytaert2004-11-29
| | | | not being present when previews are optional.
* - Patch #13263 and #13265 by arnab: added word-based truncation and made the ↵Dries Buytaert2004-11-21
| | | | comment module use it to extract subjects.
* - Patch 13180 by chx: renamed check_query() to db_escape_string() and ↵Dries Buytaert2004-11-21
| | | | | | implemtented it properly per database backend. Read the manual for pg_escape_string: "Use of this function is recommended instead of addslashes()." Or read sqlite_escape_string: "addslashes() should NOT be used to quote your strings for SQLite queries; it will lead to strange results when retrieving your data."
* - Patch #12783 by Stefan: various small consistency/usability improvements.Dries Buytaert2004-11-15
|
* - Patch #11875 by Neil Drumm: block module configuration improvements.Dries Buytaert2004-10-31
| | | | | | | | | | The primary goal of this patch is to take the 'custom' and 'path' columns of the block overview page and make them into something understandable. As of Drupal 4.5 'custom' lacked an explanation which wasn't buried in help text and path required dealing with regular expressions. Every block now has a configuration page to control these options. This gives more space to make form controls which do not require a lengthy explanation. This page also gives modules a chance to put their block configuration options in a place that makes sense using new operations in the block hook. The only required changes to modules implementing hook_block() is to be careful about what is returned. Do not return anything if $op is not 'list' or 'view'. Once this change is made, modules will still be compatible with Drupal 4.5. Required changes to core modules are included in this path. An additional optional change to modules is to implement the additional $op options added. 'configure' should return a string containing the configuration form for the block with the appropriate $delta. 'configure save' will come with an additional $edit argument, which will contain the submitted form data for saving. These changes to core modules are also included in this patch.
* - Patch #12232 by Steven/UnConed: search module improvements.Dries Buytaert2004-10-31
| | | | | | | | | | | | | | | | | | | | | | 1) Clean up the text analyser: make it handle UTF-8 and all sorts of characters. The word splitter now does intelligent splitting into words and supports all Unicode characters. It has smart handling of acronyms, URLs, dates, ... 2) It now indexes the filtered output, which means it can take advantage of HTML tags. Meaningful tags (headers, strong, em, ...) are analysed and used to boost certain words scores. This has the side-effect of allowing the indexing of PHP nodes. 3) Link analyser for node links. The HTML analyser also checks for links. If they point to a node on the current site (handles path aliases) then the link's words are counted as part of the target node. This helps bring out commonly linked FAQs and answers to the top of the results. 4) Index comments along with the node. This means that the search can make a difference between a single node/comment about 'X' and a whole thread about 'X'. It also makes the search results much shorter and more relevant (before this patch, comments were even shown first). 5) We now keep track of total counts as well as a per item count for a word. This allows us to divide the word score by the total before adding up the scores for different words, and automatically makes noisewords have less influence than rare words. This dramatically improves the relevancy of multiword searches. This also makes the disadvantage of now using OR searching instead of AND searching less problematic. 6) Includes support for text preprocessors through a hook. This is required to index Chinese and Japanese, because these languages do not use spaces between words. An external utility can be used to split these into words through a simple wrapper module. Other uses could be spell checking (although it would have no UI). 7) Indexing is now regulated: only a certain amount of items will be indexed per cron run. This prevents PHP from running out of memory or timing out. This also makes the reindexing required for this patch automatic. I also added an index coverage estimate to the search admin screen. 8) Code cleanup! Moved all the search stuff from common.inc into search.module, rewired some hooks and simplified the functions used. The search form and results now also use valid XHTML and form_ functions. The search admin was moved from search/configure to admin/search for consistency. 9) Improved search output: we also show much more info per item: date, author, node type, amount of comments and a cool dynamic excerpt à la Google. The search form is now much more simpler and the help is only displayed as tips when no search results are found. 10) By moving all search logic to SQL, I was able to add a pager to the search results. This improves usability and performance dramatically.
* #12128: Change label for comment subject settings to match behaviour;Steven Wittens2004-10-28
|
* Comment deleting brokenSteven Wittens2004-10-18
|
* - Patch #11670 by TDobes: comment.module did not properly set a breadcrumb ↵Dries Buytaert2004-10-17
| | | | for creating/previewing new comments. The node on which your commenting and its location within the menu is not displayed. Instead, there is only a "Home" link. This is especially apparent when the "add new comment" form is set up to display on a separate page.
* - Patch #10664 by TDobes: fixed collapsed comments + updated the code comments.Dries Buytaert2004-10-16
|
* - Patch #9252 by TDobes: admins should be able to edit anonymous commenters' ↵Dries Buytaert2004-10-15
| | | | names.