Commit Graph

321 Commits (410f3010060dcb5c25f7f52f6d96aeaa9915d8ef)

Author SHA1 Message Date
Dries Buytaert df6fe90286 - Updated two watchdog calls. 2005-01-09 10:06:26 +00:00
Dries Buytaert 64b100d19a - Patch #13260 by UnConeD: watchdog module improvements.
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.
2005-01-09 09:22:40 +00:00
Dries Buytaert 1dbe0dc2ee - Patch #14917 by UnConeD/Steven:
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.
2004-12-31 09:30:12 +00:00
Dries Buytaert 09c55017f6 - Patch #14882 by Jeremy: removed some cruft: status = 2 is no more. 2004-12-29 18:41:52 +00:00
Dries Buytaert b2e04ead89 - Patch #14710 by tangent: only show post button if preview is optional or 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. 2004-12-23 06:47:24 +00:00
Steven Wittens e7b0b227b9 Making comment moderation query PGSQL/ANSI compatible. 2004-12-20 02:05:00 +00:00
Dries Buytaert 2b17b3a966 - Patch #13907 by Neil: less ways to set the page title.
* 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.
2004-12-15 21:19:42 +00:00
Dries Buytaert 260f886699 - Fixed typo: db_queryd() -> db_query() 2004-12-11 12:26:40 +00:00
Dries Buytaert d61e7a663e - Bugfix: make sure to update the node_comment_statistics table when a node is updated. 2004-12-11 11:41:17 +00:00
Dries Buytaert 60352821bf - Refactored the queue module: removed the queue module's field from the 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.
2004-12-07 16:55:38 +00:00
Dries Buytaert e4a3b6f89f - Patch #13816 by Neil: tidied up the comment module code a bit. 2004-12-02 20:24:53 +00:00
Dries Buytaert 95610bdd6c - Patch #13647 by Goba:
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.
2004-11-29 17:52:35 +00:00
Dries Buytaert e35137a5ad - Patch #12555 by svemir: fixed problem with the 'Preview comment' button not being present when previews are optional. 2004-11-29 12:59:21 +00:00
Dries Buytaert b442fad027 - Patch #13263 and #13265 by arnab: added word-based truncation and made the comment module use it to extract subjects. 2004-11-21 20:17:44 +00:00
Dries Buytaert fa97839088 - Patch 13180 by chx: renamed check_query() to db_escape_string() and 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."
2004-11-21 08:25:17 +00:00
Dries Buytaert 9979aceab0 - Patch #12783 by Stefan: various small consistency/usability improvements. 2004-11-15 11:16:39 +00:00
Dries Buytaert b3adcf05a3 - Patch #11875 by Neil Drumm: block module configuration improvements.
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.
2004-10-31 07:34:47 +00:00
Dries Buytaert 8daed9cbf3 - Patch #12232 by Steven/UnConed: search module improvements.
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.
2004-10-31 03:03:27 +00:00
Steven Wittens 3e406e8f40 #12128: Change label for comment subject settings to match behaviour; 2004-10-28 23:56:14 +00:00
Steven Wittens b2f8787c90 Comment deleting broken 2004-10-18 20:10:18 +00:00
Dries Buytaert a423eefe95 - Patch #11670 by TDobes: comment.module did not properly set a breadcrumb 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. 2004-10-17 19:09:09 +00:00
Dries Buytaert 7ebd772f41 - Patch #10664 by TDobes: fixed collapsed comments + updated the code comments. 2004-10-16 07:22:26 +00:00
Dries Buytaert 7cf0e9af33 - Patch #9252 by TDobes: admins should be able to edit anonymous commenters' names. 2004-10-15 16:57:20 +00:00
Dries Buytaert 49dcf84fdf - Patch #11605 by TDobes: if a user happens to have uid=0 defined in their database, the name assigned to that user will always override any name submitted by an anonymous commenter. Checking against $user->uid instead of $user->registered_name fixes this problem. 2004-10-15 11:13:11 +00:00
Dries Buytaert dfb3c3aec4 - Patch #11577 by drumm: changed t('ago') -> t('%time ago', ...) to make it
translatable in languages such as Spanish.
2004-10-15 05:10:35 +00:00
Steven Wittens 9e4399aae8 #10677: Confirmation when deleting a block + unifying confirmation screens. 2004-10-14 15:28:24 +00:00
Steven Wittens b5806c7595 #11503: Fixing bad t() 2004-10-13 20:29:12 +00:00
Steven Wittens a185443ed3 #11503: consistency improvements by Stefan 2004-10-12 16:16:22 +00:00
Steven Wittens 2c4a82bda2 #11503: (more) missing t() 2004-10-12 16:10:54 +00:00
Dries Buytaert 5407b630e3 - Orphaned the node_link() hook as per #9769. 2004-10-10 11:12:57 +00:00
Dries Buytaert bc5ec8ea09 - Fixed critical bug #10664: made comment in collapsed mode work. One more down. 2004-10-10 10:56:59 +00:00
Dries Buytaert f8e532f714 - Patch #4925 by erikhopp: comment module emits too many id="comment"s, so the comment form's one has been renamed to 'comment-form'. This is also more consistent with the 'node-form' one. 2004-10-09 06:43:50 +00:00
Steven Wittens bd7d38bd5b Hopefully fixing all filter issues with comments. 2004-10-07 14:33:54 +00:00
Dries Buytaert f9b92e0f68 - Patch #11316 by ankur: added missing %d in INSERT-query. 2004-10-05 21:42:58 +00:00
Steven Wittens c539391656 Fixing problems with filter formats and comments. 2004-10-05 19:45:32 +00:00
Dries Buytaert 407af36656 - Patch #11286 by Neill: fixed broken link in form description. 2004-10-05 05:08:01 +00:00
Dries Buytaert f97999d1a4 - Patch #10920 by jhriggs: fixed bug with comment moderation. 2004-09-30 18:13:25 +00:00
Steven Wittens 1298d2b9c9 Modified version of #10230: Put placement of filter format selector in a module's hands, and move it below the relevant textarea. 2004-09-28 19:13:03 +00:00
Dries Buytaert fc3676d50b - Patch #10977 by Gerhard: added node permission checks to the SQL queries.
NOTE: I had to modify the book module patch for it to work -- it was throwing SQL errors at me.
2004-09-27 20:10:29 +00:00
Dries Buytaert 002f6c9361 - Patch #10722 by drumm: the user's signature was not remembered when the 'my account' form was found to be invalid. 2004-09-20 20:02:42 +00:00
Dries Buytaert 5c7983c4de - Patch #8179 by JonBob: reintroduced menu caching. 2004-09-16 07:17:56 +00:00
Dries Buytaert 1606163718 - Fixed SQL problem in _comment_update_node_statistics(). 2004-09-14 20:29:34 +00:00
Dries Buytaert c09158c3f3 - Patch #10308 by Bart Jansens/ccourtne: fixed shadow copies.
- Patch #10308 by ccourtne: performance improvements: comment statistics are now cached in a new SQL table which significantly improves performance of the forum block and the forum pages.  These pages are about 3 times faster now!
2004-09-14 05:48:02 +00:00
Dries Buytaert 841d5a0cf8 - Bugfix: comment subject where not always enabled by default because the default value of comment_subject_field varies. 2004-09-11 13:45:23 +00:00
Steven Wittens 42f1c0c93c Comment.module:
Comment previewing was broken since 1.263, resulting in a double page.
2004-09-05 10:29:38 +00:00
Steven Wittens 51cf18e531 - #9292: Make Drupal (somewhat) PHP5 compatible. xtemplate is still horribly broken. 2004-08-22 17:03:42 +00:00
Dries Buytaert 94e30bf776 - Patch by JonBob: for consistency and readability, add brief descriptions of each source file inside the @file comment block at the head of the file. This helps with Doxygen indexing, and also allows neophytes to see what a file does immediately on opening the source, regardless of the organization of the hooks. 2004-08-21 06:42:38 +00:00
Dries Buytaert fa25c7a0ca - Code improvements by Stefan: use capital letters for header titles (and added some missing t() functions). 2004-08-19 15:41:57 +00:00
Dries Buytaert 83a739bd89 - Code improvements by Stefan: made all status messages consistent (and easier to translate). 2004-08-18 19:57:27 +00:00
Dries Buytaert 2f33b939c5 - Patch #9983 by Stefan: usability improvement: made sure all status messages start with a capital letter. 2004-08-16 18:02:48 +00:00