Drupal 6.24

Merge branch '6.22-security' into 6.x

Drupal 6.23

Issue #879270 by Ben Coleman: query in taxonomy_node_get_terms() needs the v.weight field added to the SELECT because it was already present in the ORDER BY; improved PostgreSQL compatibility

Issue #12274 by markoshust, DamienMcKenna, seanbfuller, cburschka, aufumy: do not accept email addresses with dots at the end as valid

Issue #600836 by tim.cosgrove, dww, naxoc, Dave Reid: prevent batches from going indefinitely if their 'finished' value becomes bigger than 1

Issue #289504 by mikeryan, catch, moshe weitzman: backport indexes from Drupal 7 on comments and node_comment_statistics to improve performance of mass-user operations such as deleting users en masse

Issue #1278160 by tim.plunkett: lower the severity level in watchdog() for orphaned actions

Issue #975754 by jhodgdon, Albert Volkman: fix phpdoc about access callback inheritance for menu items

Issue #336483 by brianV, catch: add index for comment_count on column in node_comment_statistics to avoid full table scan when search index is updated for example

Issue #1137848 by pillarsdotnet, BrockBoland, tim.plunkett: hide /filter/tips page from well behaving search engines, it is pointless to be included in search indexes

Issue #648516 by cafuego, williamestrada: node_build_filter_query() should consider the vid of nodes when joining on term_data to consider only current revisions of nodes

Issue #767104 by mgriego, EugenMayer, mkalkbrenner: taxonomy_node_save() should refresh the node terms (especially for free tagging vocabularies) when nodes are updated, so they are kept up to date in the same HTTP request

Issue #720876 by pillarsdotnet: user_register_validate() should invoke user_module_invoke() with an account object rather than an array; improved PHP 5.3 compatibility

Issue #269877 by neochief, roderik, akaliel, mathieu, jct, Jo Wouters, fago, ufku, fmjrey, nonsie, pfournier: avoid duplicate paths (again) in path_set_alias()

Issue #147000 by Berdir, Dave Reid, pwolanin, pillarsdotnet, TR, jbrauer, dstol: rework module_rebuild_cache() and system_theme_data() to support locking, so that concurrent requests will not corrupt the data; fixes an issue where themes got disabled occasionally

Issue #156582 by c960657, drico, pwolanin, townxelliot, Damien Tournoud, kbahey, mikeytown2, jpmckinney, craig_, zyxware: add missing timeout support to drupal_http_request() to avoid Drupal hanging on web service requests

Issue #556842 by mh86, catch, bangpound, wojtha, deviantintegral: optimize taxonomy_get_tree() by building the tree internally instead of recursively to improve performance; especially good for bigger taxonomies

Issue #504506 by andypost, pillarsdotnet, podarok: Drupal.formatPlural() did not pass on extra arguments for translations of some plural values

Issue #1173012 by carlos8f, Fabianx, G�bor Hojtsy: do not remove blocks for disabled modules in _block_rehash() to avoid blocks disappearing for people following the update guide to the letter disabling contrib modules when updating Drupal core

Issue #883038 by pillarsdotnet, ShutterFreak, anarcat: ereg() and ereg_replace() are deprecated but Drupal 6 will not migrate to preg to avoid breaking APIs, so we need to silence the warnings coming out of the use of ereg on PHP 5.3+

Issue #1174496 by Eric_A, grendzy, greg.1.anderson, c31ck: drupal_error_handler() should not assume filter_xss() is available. It might be called before that is loaded. Also fixes Drush compatibility.

Issue #825972 by daniels220, Reidsy, Albert Volkman: add phpdoc documentation to sess_read() and sess_write()

Issue #1260932 by DjebbZ, pwolanin: hook_nodeapi() optional arguments should not be made required by some core modules either

Issue #208793 by Damien Tournoud, kbahey, TR: instead of hardcoding the HTTP protocol version in responses, respond with the HTTP protocol version requested (or fall back to HTTP/1.0 if the requested protocol/version is unknown)

Issue #971812 by mfb, twistor: aggregator ignores Atom id element, which is equivalent to RSS guid element

Issue #845542 by pounard: db_query_temporary() will fail without CREATE TEMPORARY TABLES permission on MySQL, update install documentation for that

Issue #1225364 by sivaji, �ull: body field label is configurable but hardcoded in min word count validation error message

Issue #1103310 by c960657: lock_acquire() does not properly extend expired lock due to misuse of db_query() to check for affected rows

Issue #909274 by ergonlogic, franz: user/0/delete should not be accessible even for administrators

Issue #1006938 by dww, westwesterson: avoid PHP warnings in the PHP 4 XML parser caused by the relatively new XML data pieces produced at updates.drupal.org

Issue #480082 by pp: add previously undocumented variables in system_send_email_action_form()'s message field description

Issue #1173012 by carlos8f, Fabianx: fix poll_block() to not wrap the block list operation in permission checks which could result in block placement and configuration lost when block rehashing is run with a user lacking permissions to view content

Issue #1283442 by Ben Coleman, twistor, ggevalt: aggregator_block()'s permission check on the list, configure and save operations could result in loss of aggregator blocks when block rehasing is invoked with a user who has no access to feeds; permissions should only be checked on the view op

Issue #1169034 by mfb: fix notice in file_create_filename() for cases when the ext variable did not exist previously

Issue #1196102 by James_Stallings, noisetteprod: improve phpdoc on taxonomy_check_vocabulary_hierarchy()

Issue #338402 by rocket_nova, jhodgdon: improve documentation on node_get_types()

Issue #1098556 by barbi, rocket_nova, kiamlaluno: search_box() documentation referred to theme_search_box_form() which is in fact implemented in search-block-form.tpl.php in Drupal 6

Issue #1192178 by pillarsdotnet, jhodgdon: fix user_module_invoke() argument names to make more sense and phpdoc to document them (no behavioral change of course)

Issue #1288838 by anrikun: fix bad code indentation in _locale_batch_build()

Issue #1175614 by barbi, oriol_e9g, CashWilliams, aenw: improve phpdoc on block_list()

Issue #1100260 by barbi, jhodgdon: fix phpdoc in batch operations example to have a redirect

Issue #372766 by jhodgdon, jn2: phpdoc improvements to user_save()

Issue #1222802 by franz, droplet, jhodgdon: cleanup for pager phpdoc blocks; add missing text argument doc to theme_pager_link

Issue #1190846 by vitalsouto, ninizik: remove one of two copies of repeated phpdoc documentation on actions

- Patch #300714 by j0hn-smith, alexjarvis, lilou, linuxbox: sort node revisions by vid not timestamp.

Issue #1149628 by Ben Coleman: fix blog module query to include all ORDER BY fields in select for PostgreSQL compatibility

Issue #201876 by gpk: improve yellow in Garland tabledrag to stand out better for usability

Issue #356359 by mgifford, Agileware: improve accessibility of the search form

Issue #925580 by tbenice, rsevero, marcp: dix upload modules's form altering to be clearly compatible with PHP 5.3

Issue #188947 by lyricnz: date_validate() takes a form element, not a whole form, correct the variable name to clean this up

Issue #1154722 by mattconnolly, barbi, jhodgdon: link Form API reference to Drupal 6 version in Form API docs

Issue #1095066 by drewish, barbi: improve node_build_content() return value documentation

Issue #401266 by pillarsdotnet, jhodgdon: improve documentation on user_mail_tokens() to be more accurate on required account properties

Issue #1184106 by barbi: improve documentation on db_set_active()

Issue #1063636 by barbi, Rob Loach, TR: reference drupal_add_css() in drupal_get_css() docs

Issue #1176524 by andypost, TR, dww: remove leftover CVS ID tags from Drupal core

Issue #1169080 by barbi, csdco: improve documentation of file_set_status()

Issue #623276 by jhodgdon: improve documentation on file_scan_directory()

Issue #1123120 by kiamlaluno, mr.baileys: improve documentation for theme preprocess; hook argument was missing

Now onto Drupal 6.23-dev

Drupal 6.22

Drupal 6.21

Issue #113611 by crischan, jjrey, JuliaKM, salvis, Offlein: Forum post counts incorrect when using node access control modules

Issue #321063 by eMPee584, Kars-T, pillarsdotnet: Path book/export/html/N with a non-book nid should abort and return 404 not found.

Issue #235673 by yched, Damien Tournoud, swentel, pwolanin, redndahead, sun, carlos8f, c960657: Changes to block caching mode not caught

Issue #384794 by trond, mvc, alduya, intuited, jaydublu, jhedstrom: trim() .po file comments on import, so that custom textgroup imports will find their right database equivalent

Issue #163445 by webengr, mlsamuelson, greg.1.anderson, pescetti: fix issues with IE not being able to download private files served over SSL with additional headers

Issue #118108 by Chris Johnson, cburschka, bleen18: remove node_access table entries when a node is deleted to avoid leftover cruft

Issue #632080 by donquixote, mathieu, jonhattan, andypost: clone return values from _system_theme_data(), so modules careless about lack of object copies will not overwrite the system level cached values

Issue #107051 by webchick, gdevlugt, vladimir.dolgopolov, swentel, Dave Reid, tuffnatty, zyware, crifi, tobiasb: Fix for users showing multiple times in the who's online block

Issue #292790 by ag888, kjay, mr.baileys, recidive, malc0mn, jonathan1055, null: fix menu machine name validation form API constuct, so we don't need to validate it with custom code

Issue #307636 follwup by quicksketch: fix to make image_get_info() on IIS work better had a variable name typo before; now fix that

#1093352 by ohnobinki: fix invalid HTML entity in batch.inc

Issue #905576 by droplet, idflood, tim.plunkett: /contact should not be blocked from search engines

Issue #335621 by Dave Reid, jrguitar21, jhodgdon: fix PHP strict warning by initializing object in template_preprocess_block_admin_display_form()

Issue #977174 by AlexisWilke: Fix minor HTML validity issue in update.php when generating list of results

Issue #690358 by sbrattla, jhodgdon, nenne: overhaul file_check_directory() documentation due to various issues

Issue #1013986 by disturbedmime, jhodgdon: fix unfinished sentence in file_validate_extensions()

Issue #480152 by jhodgdon: filter_filter() docs were incorrect and outdated

Issue #470198 by figaro, ashmiler: fix small typo in menu item description for admin/build/menu

Issue #519362 by mfb, bfoehle: Resolve dc:creator tag as the item author if author tag is not set in aggregator module.

Issue #1105384 by claar: fix system_admin_menu_block_page() first doc line

Issue #1087092 by danillonunes, drewish: fix a tiny misues of @result in drupal_map_assoc() documentation

Issue #1089332 by jeffschuler: fix misuse of @see in db_add_field() documentation

Issue #1103938 by kiamlaluno, jhodgdon, jeffschuler: fix misues of @name in various include files and document theme.inc constants better

Stripping CVS keywords

Now onto Drupal 6.21-dev

Drupal 6.20

#986682 by pkiraly: improve code documentation for db_table_exists() and db_column_exists()

Roll back #147000, prevented Drupal from being installed.

#307636 by zbricoleur, sreynen, quicksketch: fix file identification bug with image file processing on Microsoft IIS

#212130 by salvis, boydjd, Steven, grendzy, Damien Tournoud: more complete support for unicode entities, to account for previously missing entities in decode_entities()

#772678 by sun, jpmckinney, Berdir, markus_petrux: no way to specify default collation, entirely depended on database configuration (which might be inappropriate)

#881132 by HLopes, Garrett Albright: CSS files with non-UTF-8 characters broke CSS optimization

#488166 by EmanueleQuinto, Damien Tournoud, jhodgdon: search relevance calculation fails if last_comment_timestamp is NULL

#422218 by salvis, jeremiah.snapp: fix a case in forum module where non forum tids might get picked as the forum topic tid

#357785 by arnoldc, gravalsyr, miro_dietiker, plach: retain the tnid value for new nodes saved, so the node object reflects the database

#917670 by mr.baileys, rdrh555: fix documentation for drupal_alter()

#287647 by bjaspan, lilou, mikejoconnor, cafuego, D�ja: cast invalid hook_schema() results into arrays at all times