Commit message (Collapse) | Author | Age | |
---|---|---|---|
* | #331951 by aspilicious, sun, jhodgdon, Morbus, et al: Figure out and apply ↵ | Angie Byron | 2010-05-06 |
| | | | | coding standard for casting. | ||
* | - Patch #716496 by JohnAlbin: documentation updates for theme functions. | Dries Buytaert | 2010-04-13 |
| | |||
* | - Patch #520764 by Damien Tournoud, JoshuaRogers, brianV: fixed ↵ | Dries Buytaert | 2009-12-05 |
| | | | | SA-CORE-2009-007: request values in URL, including password/username. | ||
* | - Patch #648744 by Crell: specified more @return type definitions for ↵ | Dries Buytaert | 2009-12-04 |
| | | | | auto-completion in IDEs. | ||
* | - Patch #266434 by catch, merlinofchaos, sun: fixed E_NOTICE in ↵ | Dries Buytaert | 2009-11-14 |
| | | | | pager_load_array(). | ||
* | - Patch #321023 by Everett Zufelt, mgifford: improve accesibility in pager.inc. | Dries Buytaert | 2009-10-31 |
| | |||
* | - Patch #572618 by effulgentsia, pwolanin, sun: all theme functions should ↵ | Dries Buytaert | 2009-10-09 |
| | | | | take a single argument. Code clean-up and performance improvement. Woot. | ||
* | - Patch #578520 by sun | c960657, mfb, Dries, catch, mattyoung: make in ↵ | Dries Buytaert | 2009-09-29 |
| | | | | url() only accept an array. Another nice API clean-up! | ||
* | #564394 by Berdir and Crell: Removed database BC layer. nah nah nah nah... ↵ | Angie Byron | 2009-09-18 |
| | | | | hey hey hey... gooood byeeee... | ||
* | #496516 by Crell and Berdir: Moved query_alter() into a preExecute() method, ↵ | Angie Byron | 2009-08-26 |
| | | | | so that modules can know the final query/arguments before they are run. | ||
* | #326539 by JohnAlbin, sun, cha0s, ultimateboy, Rob Loach, Damien Tournoud: ↵ | Angie Byron | 2009-08-22 |
| | | | | Convert 'class' attribute to use an array, not a string. | ||
* | #514914 by Gábor Hojtsy: Add back removed pager_limits variable. | Angie Byron | 2009-08-11 |
| | |||
* | - Patch #491556 by Berdir: completed converting core to DBTNG. Oh my. Kudos ↵ | Dries Buytaert | 2009-07-28 |
| | | | | to Berdir for this humongous effort. | ||
* | - Patch #430904 by Berdir: fixed notice in case there are two pagers on one ↵ | Dries Buytaert | 2009-05-12 |
| | | | | page. | ||
* | - Patch #330748 by stBorchert: remove from theme_pager*. | Dries Buytaert | 2009-04-26 |
| | |||
* | - Patch #299267 by Crell: add extender support to the SELECT query builder. | Dries Buytaert | 2009-02-22 |
| | |||
* | #282405 by Damien Tournoud, lilou, Dave Reid: Enforce coding standard on elseif. | Angie Byron | 2008-10-12 |
| | |||
* | #196667 (GHOP 45) by fberci: add '@ingroup themeable' to all themeable functions | Gábor Hojtsy | 2007-12-06 |
| | |||
* | - Patch #163508 by profix898: fixed 4 E_ALL notices. | Dries Buytaert | 2007-08-02 |
| | |||
* | - Patch #135931 by EclipseGc: semantic update for core pager. | Dries Buytaert | 2007-05-01 |
| | |||
* | - Patch #111347 by Steven: refactor url() and l(). | Dries Buytaert | 2007-02-15 |
| | |||
* | - Patch #87995 by merlinofchaos: added missing css. | Dries Buytaert | 2006-10-15 |
| | |||
* | - Patch #80934 by timnc: more t() fixes. | Dries Buytaert | 2006-08-27 |
| | |||
* | #80200 by gorgen. More strict pager query regexp. | Neil Drumm | 2006-08-24 |
| | |||
* | - Patch #78364 by chx: remove pager cruft. | Dries Buytaert | 2006-08-16 |
| | |||
* | - Patch #72204 by nedjo: upper-cased all TRUE/FALSE/NULL constants. | Dries Buytaert | 2006-07-05 |
| | |||
* | #5371, drupal_get_destination, pager and tablesort array handling, patch by ↵ | Gerhard Killesreiter | 2006-04-13 |
| | | | | Steven | ||
* | - Patch #44771 by jvandyk: small performance improvement. | Dries Buytaert | 2006-01-15 |
| | |||
* | - #44498: Clean up pager / make more accessible | Steven Wittens | 2006-01-15 |
| | |||
* | - Patch #44498 by m3vrck: improved HTML generated code for pagers. | Dries Buytaert | 2006-01-14 |
| | |||
* | - Patch #30930 by m3avrck/deekayen: cured PHP5 warnings. | Dries Buytaert | 2005-10-22 |
| | |||
* | - #32603: Clean up theme_pager_link (drumm) | Steven Wittens | 2005-10-21 |
| | |||
* | - Patch #29385 by chx: no ?> add end of files. | Dries Buytaert | 2005-08-25 |
| | |||
* | - Patch #27980 by Neil Drumm: removed unused function. | Dries Buytaert | 2005-08-10 |
| | |||
* | - #24673: Fix deprecated usage of implode | Steven Wittens | 2005-07-30 |
| | |||
* | - Fix mistakes in pager patch. | Steven Wittens | 2005-05-25 |
| | |||
* | - #23495: Clean up pager code. Now uses $page instead of $from, and counts ↵ | Steven Wittens | 2005-05-25 |
| | | | | pages, not items. | ||
* | - #18817: Clean up plain-text checking (see drupal-devel!) | Steven Wittens | 2005-03-31 |
| | |||
* | - Patch by Remco: <div> -> </div>. | Dries Buytaert | 2005-01-28 |
| | |||
* | - Patch by Jeremy: fixed unclosed " | Dries Buytaert | 2005-01-27 |
| | |||
* | - Patch by Jeremy: made the diffs more meaningful. | Dries Buytaert | 2005-01-27 |
| | |||
* | - Patch #16273 by Jeremy: improved the themability of the pager. | Dries Buytaert | 2005-01-27 |
| | |||
* | Pager_query's count query was broken if no query arguments were given. | Steven Wittens | 2004-12-06 |
| | |||
* | - Patch by Steven: fixed bug in pager_query(). | Dries Buytaert | 2004-12-04 |
| | |||
* | Fix for pager_query() after #13581 (array of query arguments). | Steven Wittens | 2004-12-02 |
| | |||
* | - Patch #13581 by Steven: Db_query() allows a variable amount of parameters ↵ | Dries Buytaert | 2004-11-29 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | so you can pass the query arguments in. There is however an alternative syntax: instead of passing the query arguments as function arguments, you can also pass a single array with the query arguments in it. For example the following two statements are equivalent: db_query($query, $a, $b, $c); db_query($query, array($a, $b, $c)); This usage is particularly interesting when the query is constructed dynamically, and the amount of arguments to pass varies. In that case we use the second method to avoid using call_user_func_array(). This behaviour is not documented explicitly, but it is used in several places. However, db_query_range() and pager_query() do not support this syntax properly, which means there are several pieces of code which still revert to the ugly call_user_func_array() call. This patch updates db_query_range() and pager_query() so they support the array-passing method. I also added documentation about this method to each of the db functions. I also cleaned up the code for db_query (it was weird and hard to understand) and moved db_query() and db_queryd() from database.xxxxx.inc to database.inc: it was the same between both mysql and pgsql, as it doesn't do anything database specific. It just prefixes the tables and inserts the arguments. The actual db query is performed in _db_query(), which is still in database.xxxxx.inc. Finally, I updated several places with the new syntax, and the code is a lot cleaner. For example: - array_unshift($params, "SELECT u.* FROM {users} u WHERE $query u.status < 3"); - $params[] = 0; - $params[] = 1; - $result = call_user_func_array('db_query_range', $params); + $result = db_query_range("SELECT u.* FROM {users} u WHERE $query u.status < 3", $params, 0, 1); and - return call_user_func_array('db_query_range', array_merge(array($query), $args, array((int)$pager_from_array[$element], (int)$limit))); + return db_query_range($query, $args, (int)$pager_from_array[$element], (int)$limit); I've tested it on mysql. I didn't alter the actual db behaviour, so pgsql should be okay too. This patch is important because many people avoid the call_user_func_array() method and put data directly into the db query. This is very, very bad because the database prefix will be applied to it, and strip out braces. It's also generally bad form as you have to call check_query() yourself. With the new, documented syntax, there is no more excuse to put data directly in the query. | ||
* | - Patch #7161 by jhriggs: fixed probem with 'last page' link not being ↵ | Dries Buytaert | 2004-10-31 |
| | | | | correct under certain circumstances. | ||
* | - Patch #10663 by JonBob: documentation improvements: fixed some typos and ↵ | Dries Buytaert | 2004-09-09 |
| | | | | improved consistency to the use of Doxygen/api.module commands in the comments. | ||
* | - Patch #7535 by Gerhard: one could not search for the word 'From'. | Dries Buytaert | 2004-08-12 |
| | |||
* | - Patch #9478 by JonBob: allow printf-style arguments in pager_query. | Dries Buytaert | 2004-07-25 |
| | | | | | | Currently pager_query() is the black sheep of the database query family, because it does not allow for printf-style arguments to be inserted in the query. This is a problem because it introduces developer confusion when moving from an unpaged query to a paged one, and it encourages substitution of variables directly into the query, which can bypass our check_query() security feature. This patch adds this ability to pager_query(). The change is backwards-compatible, but a couple calls to the function in core have been changed to use the new capability. |