Audean Wiki

User Tools

Site Tools

You are here: Audean Wiki for Mediacore services » Drupal at OSInet » Zeitgeist patch
Trace:
dr:zeitgeist

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revisionPrevious revision
Next revision		Previous revision
dr:zeitgeist [2005/11/30 21:19] – Implemented in v2 of the patch fgmdr:zeitgeist [2020/11/23 17:23] (current) – external edit 127.0.0.1
Line 4: Line 4:
  
 Ensuing discussion on [[irc://irc.freenode.net/drupal|#drupal]] and on the patch page suggested that: Ensuing discussion on [[irc://irc.freenode.net/drupal|#drupal]] and on the patch page suggested that:
 +
  
   * the idea was interesting   * the idea was interesting
Line 11: Line 12:
  
 As this involves changes to core features, I hope developers with a deeper understanding of core than me will comment on this/edit this page to improve it before I write code involving these core changes. The wiki structure allows this more easily than comments on the patch page. As this involves changes to core features, I hope developers with a deeper understanding of core than me will comment on this/edit this page to improve it before I write code involving these core changes. The wiki structure allows this more easily than comments on the patch page.
 +
  
 ===== Problems with the current patch ===== ===== Problems with the current patch =====
Line 16: Line 18:
 If you look at the patch, what it basically does is this, along with the definition of a new block in search_block: If you look at the patch, what it basically does is this, along with the definition of a new block in search_block:
  
-<php>function _search_recent()+<code php> 
 +function _search_recent()
   {   {
   $sq = "   $sq = "
Line 37: Line 40:
   unset ($ar);   unset ($ar);
   return $output ;   return $output ;
-  }</php>+  } 
 +</code>
  
 You'll notice the "substr" line is dependent of the current format of the watchdog entries, which means it will most likely break on any translated site. And I know drupal style does not like multiline queries. You'll notice the "substr" line is dependent of the current format of the watchdog entries, which means it will most likely break on any translated site. And I know drupal style does not like multiline queries.
  
 Also, processing is done in PHP whereas a better implementation would just ask the DBMS for the set of distinct entries in proper order to minimize the global workload. Also, processing is done in PHP whereas a better implementation would just ask the DBMS for the set of distinct entries in proper order to minimize the global workload.
- 
  
 ===== Changes to watchdog ===== ===== Changes to watchdog =====
  
-  * Implemented in v2 of the patch: add a new column (say, "data") to the watchdog table, to hold a php serialized content, or NULL +  * Implemented in v2 of the patch:  
-  * Implemented in v2 of the patch: modify bootstrap.inc/watchdog to pass a new parameter: +    * add a new column (say, "data") to the watchdog table, to hold a php serialized content, or NULL 
- +    * modify bootstrap.inc/watchdog to pass a new parameter: 
-<php>+<code php>
 function watchdog( function watchdog(
   $type,    $type, 
Line 55: Line 58:
   $severity = WATCHDOG_NOTICE,    $severity = WATCHDOG_NOTICE, 
   $link = NULL,   $link = NULL,
-  $data = NULL) {</php>+  $data = NULL) { 
 +</code>
  
 The new parameter could be used by modules aware of this new feature to store content of their choice in the watchdog along with the standard parameters, while retaining compatibility with existing call through the use of a default NULL. The new parameter could be used by modules aware of this new feature to store content of their choice in the watchdog along with the standard parameters, while retaining compatibility with existing call through the use of a default NULL.
Line 66: Line 70:
  
 In the current 4.6.x core, only watchdog module queries the watchdog table, and does so with queries like: In the current 4.6.x core, only watchdog module queries the watchdog table, and does so with queries like:
 +
 <code sql>select w.* [...] from {watchdog} w</code> <code sql>select w.* [...] from {watchdog} w</code>
-which means no outside modules should be impacted. These queries could be improved to avoid querying this additional column if it is not needed. 
  
 +which means no outside modules should be impacted. These queries could be improved to avoid querying this additional column if it is not needed.
  
 ===== Changes to search ===== ===== Changes to search =====
  
 +  * Implemented in v2 of the patch: 
 +    * searches beyond "node" and "user" could be displayed too: this mechanism could be used by modules implementing hook_search, since the content, as a serialized object, could also store additional information for these modules.
 +    * with this information available, recent searches could be displayed by unserializing these data instead of processing the HTML-formatted contents of the current columns, allowing full i18n compatibility.
   * having this new column available would allow search to store more usable information in the watchdog, typically by storing a serialized version of the query stats:   * having this new column available would allow search to store more usable information in the watchdog, typically by storing a serialized version of the query stats:
     * term searched     * term searched
     * result count     * result count
     * something else ?     * something else ?
-  * with this information available, recent searches could be displayed by unserializing these data instead of processing the HTML-formatted contents of the current columns, allowing full i18n compatibility. 
   * as suggested by Boris Mann, having this more resilient content would allow for this feature to serve better, possibly by creating a RSS feed on recent searches   * as suggested by Boris Mann, having this more resilient content would allow for this feature to serve better, possibly by creating a RSS feed on recent searches
-  * the current display, limited to a fixed count of recent searches, could be improved tby the addition of paging and optional display of the stats attached to the search +  * the current display, limited to a fixed count of recent searches, could be improved by the addition of paging and optional display of the stats attached to the search
-  * searches beyond "node" and "user" could be displayed too: this mechanism could be used by modules implementing hook_search, since the content, as a serialized object, could also store additional information for these modules. +
  
 ===== TODO ===== ===== TODO =====
 +
 2005-11-29 8:33 chx says: 2005-11-29 8:33 chx says:
 +
 If you want to win the developers' hearts then you change the trigger_error in _db_query to a watchdog call (avoid infinite loops! keep a static var about 'tried watchdog it') which stores the debug_backtrace and lift the debug_backtrace renderer from devel. The latter is a contrib module but this would be just great, i think two lines o' code and would make drupal a hell lot easier to debug. If you want to win the developers' hearts then you change the trigger_error in _db_query to a watchdog call (avoid infinite loops! keep a static var about 'tried watchdog it') which stores the debug_backtrace and lift the debug_backtrace renderer from devel. The latter is a contrib module but this would be just great, i think two lines o' code and would make drupal a hell lot easier to debug.
  
 2005-1129 20:33 After further discussion, it looks like the idea is to log the debug_backtrace() information in the additional field. 2005-1129 20:33 After further discussion, it looks like the idea is to log the debug_backtrace() information in the additional field.
dr/zeitgeist.1133385563.txt.gz · Last modified: 2020/11/23 17:23 (external edit)