- Development - lists.drupal.org

[drupal-devel] Release critical bugs for August 04, 2005
by drupal-devel＠drupal.org 04 Aug '05

04 Aug '05

/ bug reports email spam filtering does not "see" email addresses when they are NOT preceded by a word age: 28 weeks 4 days url: http://drupal.org/node/15654 / bug reports Very serious bug in node display age: 15 hours 38 min url: http://drupal.org/node/28116 / bug reports Blank page when doing user login directly by using user/login URL age: 2 days 8 hours url: http://drupal.org/node/27996 404 sites shown twice age: 2 days 8 hours url: http://drupal.org/node/27990 js addLoadEvent not working. age: 4 days 1 hour url: http://drupal.org/node/27884 Bug in forum.module with postgresql (?) age: 6 days 21 hours url: http://drupal.org/node/27665 Forum bug with drupal 462 age: 6 days 21 hours url: http://drupal.org/node/27663 enable MySQL client side for UTF8 age: 2 weeks 2 days url: http://drupal.org/node/26990 Drupal 4.6.2 Doesn't allow login (IIS 5.1, PHP 5.0.4) assigned: bigeye age: 2 weeks 5 days url: http://drupal.org/node/26780 Problem in the 4.6.2 upgrade - node.module age: 4 weeks 2 days url: http://drupal.org/node/26342 postgresql autocomplete code age: 5 weeks 8 hours url: http://drupal.org/node/26027 Accidentally Deleting Forums Vocabulary in Categories Breaks Forums? age: 8 weeks 5 days url: http://drupal.org/node/24274 Taxonomy Multiple Hierarchy doesn't display children for all its parent age: 9 weeks 1 day url: http://drupal.org/node/24023 PG_QUERY ERROR - LOCK TABLES - IMPLEMENTATION FAULT. age: 11 weeks 3 days url: http://drupal.org/node/22911 UTF-8 node title gets truncated at wrong position age: 15 weeks 6 days url: http://drupal.org/node/20537 database table "term_node" not updated when removing a term from a node under editing and submitting age: 16 weeks 2 days url: http://drupal.org/node/20315 Reassigning a node to a different term does not work properly age: 16 weeks 3 days url: http://drupal.org/node/20278 4.6 aggregator showing/not interpreting HTML tags age: 17 weeks 4 days url: http://drupal.org/node/19874 New submit error, non-admin user, postgresql database assigned: adrian age: 21 weeks 1 day url: http://drupal.org/node/18552 When I delete a user, any forum in which s/he has commented also disappears. assigned: Jill6q age: 24 weeks 13 hours url: http://drupal.org/node/17428 Change to sesssion.inc violates UID database constraint age: 28 weeks 4 days url: http://drupal.org/node/15666 forum.module problem with postgresql v7.4.1 age: 29 weeks 2 days url: http://drupal.org/node/15411 PHP 5 gives error when session table has no session assigned: adschar age: 29 weeks 2 days url: http://drupal.org/node/15399 Argument NOT must be type boolean age: 37 weeks 2 days url: http://drupal.org/node/12950 Comments in approval queue still show up in tracker age: 41 weeks 4 days url: http://drupal.org/node/11647 Twin problems with comments assigned: Junyor age: 42 weeks 6 days url: http://drupal.org/node/11366 Fatal error: Call to undefined function: drupal_set_html_head() age: 49 weeks 1 day url: http://drupal.org/node/10317 user error: Duplicate entry assigned: Kjartan age: 1 year 34 weeks url: http://drupal.org/node/4428 / feature requests Filter the title to allow HTML comments age: 1 week 6 days url: http://drupal.org/node/27205 Novel Module Creation Request age: 7 weeks 3 days url: http://drupal.org/node/24856 moving pages en masse age: 17 weeks 6 days url: http://drupal.org/node/19754 Taxonomy module should use nodeapi 'load' age: 21 weeks 12 hours url: http://drupal.org/node/18631 Auto PHP memory maximisation age: 23 weeks 2 days url: http://drupal.org/node/17663 Allow named anchors to work without specifying full path to node age: 1 year 36 weeks url: http://drupal.org/node/4213 / support requests blocks age: 6 weeks 2 days url: http://drupal.org/node/25378 Problem with Forums age: 29 weeks 5 days url: http://drupal.org/node/15299 / bug reports search over fields.inc fields conceptual error age: 39 weeks 6 days url: http://drupal.org/node/12259 fields.inc screws up with multible multiselect fields age: 39 weeks 6 days url: http://drupal.org/node/12257 / feature requests event.module support of repeating/recurring events? assigned: thehunmonkgroup age: 1 year 37 weeks url: http://drupal.org/node/4122 / bug reports Menu item not showing age: 43 weeks 5 days url: http://drupal.org/node/11210 / bug reports segmentation fault related to filestore age: 1 year 44 weeks url: http://drupal.org/node/3001 / bug reports file size bug age: 7 weeks 5 days url: http://drupal.org/node/24728 / bug reports gallery.module doesn't synch passwords when they're changed age: 6 days 13 hours url: http://drupal.org/node/27702 Full screen slideshow in Gallery module is not working (erroring out on the wrong URL) age: 4 weeks 8 hours url: http://drupal.org/node/26479 Drupal session being logged out each time gallery module is called age: 5 weeks 6 days url: http://drupal.org/node/25614 Gallery error age: 14 weeks 2 days url: http://drupal.org/node/21290 Fatal Error: variable_get() age: 19 weeks 3 days url: http://drupal.org/node/19189 No Gallery Image Block age: 19 weeks 6 days url: http://drupal.org/node/19045 Mini applet can't find gallery at url / age: 20 weeks 4 days url: http://drupal.org/node/18785 Fatal error: Undefined class name 'galleryembed' age: 23 weeks 1 day url: http://drupal.org/node/17746 / support requests Can't Embed Gallery into Drupal age: 22 weeks 4 days url: http://drupal.org/node/18029 / bug reports Drupal-4.4-rc breaks registration setups and groups.module age: 1 year 20 weeks url: http://drupal.org/node/6392 / bug reports Term not created age: 5 days 19 hours url: http://drupal.org/node/27756 image module fails on dual site configuration age: 3 weeks 5 hours url: http://drupal.org/node/26657 Editing a previously uploaded and thumnailed image loses the thumbnail. age: 6 weeks 1 day url: http://drupal.org/node/25465 Cannot upload .jpgs age: 22 weeks 3 days url: http://drupal.org/node/18076 Image.Module does not know the correct filename age: 37 weeks 1 day url: http://drupal.org/node/13032 / support requests Images are there but not showing assigned: Stepfordtwit age: 1 week 12 hours url: http://drupal.org/node/27615 Image Module Permissions age: 4 weeks 3 days url: http://drupal.org/node/26280 path problems on upgrading... age: 13 weeks 3 days url: http://drupal.org/node/21725 / bug reports RSS "no element found at line 1" age: 4 weeks 5 days url: http://drupal.org/node/26185 Outdated help texts age: 1 year 1 day url: http://drupal.org/node/9692 Import doesn't like XML/RSS feeds with no Title element age: 1 year 9 weeks url: http://drupal.org/node/8128 / tasks move XML-parser into an inclde file. age: 6 weeks 1 day url: http://drupal.org/node/25439 / feature requests Notify exposes Group creation age: 1 day 3 hours url: http://drupal.org/node/28084 Notify.module ignores the true submission queue age: 1 year 40 weeks url: http://drupal.org/node/3765 / feature requests Internationalization support for PDF assigned: TheLibrarian age: 1 year 18 weeks url: http://drupal.org/node/6795 / bug reports Reminder email links don't work age: 5 weeks 5 days url: http://drupal.org/node/25677 / bug reports access control problems age: 1 week 5 days url: http://drupal.org/node/27273 Download link is wrong age: 6 weeks 2 days url: http://drupal.org/node/25363 Projects not listed and page not complete age: 9 weeks 2 days url: http://drupal.org/node/23956 Date for latest is incorrect age: 16 weeks 14 sec url: http://drupal.org/node/20485 Invalid link to release files age: 23 weeks 3 days url: http://drupal.org/node/17625 Node body not complete w/o releases age: 25 weeks 5 days url: http://drupal.org/node/16733 blank page after install age: 26 weeks 1 day url: http://drupal.org/node/16575 / support requests Help with everything - nothing seems to work in the latest build age: 36 weeks 2 days url: http://drupal.org/node/13310 / bug reports Can't get the inline tag to show series age: 12 weeks 1 day url: http://drupal.org/node/22520 / feature requests Request to generate email for all content changes/additions, including comments age: 2 weeks 4 days url: http://drupal.org/node/26824 / bug reports javascrip downloads square.gif for every category age: 24 weeks 1 day url: http://drupal.org/node/17366

1 0

[drupal-devel] [feature] Themeable profile list patch for profile.module
by ejort 04 Aug '05

04 Aug '05

Issue status update for http://drupal.org/node/20479 Post a follow up: http://drupal.org/project/comments/add/20479 Project: Drupal Version: cvs Component: profile.module Category: feature requests Priority: normal Assigned to: vfilby Reported by: vfilby Updated by: ejort -Status: active +Status: patch (code needs review) Attachment: http://drupal.org/files/issues/profile.module.theme_profile_list.patch (2.17 KB) Hi, This is a patch providing the same thing, but without taking the database code into the theme_profile_list function. I've had this running on one of my sites [1] for a while and found it necessary to output a table listing member contact details (note: there aren't many profile fields accessible by anonymous users in that example, but you get the idea). Unfortunately I never got around to separating it from my other profile.module modifications until now. Let me know what you think. Cheers, Eric [1] http://www.renegades.com.au/?q=members/contact-lists/all-men ejort Previous comments: ------------------------------------------------------------------------ Thu, 14 Apr 2005 03:16:04 +0000 : vfilby Attachment: http://drupal.org/files/issues/profile-themeable-user-list.patch (2.08 KB) Here is a patch for the profile module that provides a way to theme user lists. I simply took the last part of the giant if in profile_browse, moved it to a new function called theme_profile_list and mde profile_browse call the new function. ------------------------------------------------------------------------ Thu, 30 Jun 2005 04:52:17 +0000 : moshe weitzman looka useful ... please provide unified diff format. as per http://drupal.org/handbook ------------------------------------------------------------------------ Thu, 07 Jul 2005 10:39:32 +0000 : robertDouglass It doesn't really look suitable for theming, though, due to all the database logic. To make this really a theme function, you'd have to prepare the data into some variables that you can get by calling function A, and make a theme function B that outputs the variables. Things that can't be in theme functions include SQL and user access logic.

1 0

[drupal-devel] Drupal Installer? I have done some similar work
by NSK 04 Aug '05

04 Aug '05

Hi How's the Drupal Installer project going? If you haven't started on it you could use a small installer I made. It is available from http://nsk.wikinerds.org/won/ Just download the tar gz and look inside the setup directory. The tar gz is licensed under GPL2 (no later versions) but for the purposes of merging it with Drupal you can consider all the code in the tar gz to be licensed under GPL2 and later versions. Some screenshots are available at http://www.wikinerds.org/download/ If you use it, just add my name somewhere, like a CONTRIBUTORS.txt file and the source code. Basically what the installer does is writing any SQL to a database, without code changes. You provide the SQL in ASCII files. It supports database prefix, and it can create a MySQL user. My code is undocumented, but I can answer questions if you have any and you wish to use it. -- NSK http://portal.wikinerds.org

1 0

[drupal-devel] [feature] Parse atom feeds
by Prometheus6 04 Aug '05

04 Aug '05

Issue status update for http://drupal.org/node/13941 Post a follow up: http://drupal.org/project/comments/add/13941 Project: Drupal Version: cvs Component: aggregator.module Category: feature requests Priority: normal Assigned to: Prometheus6 Reported by: Capnj Updated by: Prometheus6 Status: patch (code needs review) Attachment: http://drupal.org/files/issues/AGGREGATOR_ATOM_CVS.txt (2.35 KB) Because the aggregator would always gather enough information todownload an Atom feed, the previous patch focused on parsing items/entries correctly. This patch adds more complete parsing of the feed's channel data (site link, subtitle/description, logo). Now all the feed and item fields will be properly updated. Prometheus6 Previous comments: ------------------------------------------------------------------------ Sun, 05 Dec 2004 23:26:00 +0000 : Capnj I've seen mention elsewhere on drupal.org but don't see a feature request, so here goes. The news aggregator is a great piece of Drupal as far as I'm concerned and I'd sure like to see it read all the sources out there, have the 'blog this' function, and also have a tool for browsers to generate a blog entry from a web page (like MT had). Is that a big enough request? LOL Thanks for all that everyone does here. Great stuff! gil ------------------------------------------------------------------------ Tue, 07 Dec 2004 17:38:31 +0000 : drumm RSS 2.0 feeds should work in 4.5.0 and earlier. ------------------------------------------------------------------------ Mon, 14 Feb 2005 16:04:32 +0000 : georgehotelling I've hacked Magpie [1] support into the Drupal aggregator, so it can support Atom. If you have a Drupal site and want the code, you can get it from http://george.hotelling.net/projects/magpie-aggregator-4.5.2.tgz [2] I'm not sure if I'm supposed to create a patch or what, but I needed this for ArborBlogs.com [3] and figured some other people would benefit from it. [1] http://magpierss.sourceforge.net/ [2] http://george.hotelling.net/projects/magpie-aggregator-4.5.2.tgz [3] http://www.arborblogs.com/ ------------------------------------------------------------------------ Tue, 15 Mar 2005 04:04:42 +0000 : Prometheus6 Attachment: http://drupal.org/files/issues/AGGREGATOR_ATOM_HEAD.txt (5.46 KB) Very nice. I hacked aggregator.module for 4.6 using your module as a guide...well, frankly by ripping out great chunks of code for my use. ------------------------------------------------------------------------ Thu, 28 Jul 2005 22:21:11 +0000 : Boris Mann Can someone roll a patch for this? Would be a good addition for 4.7 now that Atom is at 1.0. ------------------------------------------------------------------------ Fri, 29 Jul 2005 01:49:42 +0000 : Prometheus6 Attachment: http://drupal.org/files/issues/magpie_aggregator.zip (7.87 KB) A patch would require the Magpie include files and wholly restructures the way the feeds are parsed...the Magpie code would handle all the feeds, not just Atom. That said, the attached zip file has the necessary includes and a patch to CVS. ------------------------------------------------------------------------ Fri, 29 Jul 2005 03:36:59 +0000 : Steven Magpie RSS duplicates a lot of functionality that is already in Drupal. We decided a long time ago that we would not use it. Porting Atom parsing into the current aggregator can't be that hard. ------------------------------------------------------------------------ Mon, 01 Aug 2005 16:38:53 +0000 : Prometheus6 Attachment: http://drupal.org/files/issues/AGGREGATOR_ATOM_HEAD_0.txt (1.93 KB) Since I'm messing around with the aggregator module anyway... I developed this around Atom 0.3 feeds this weekend, but I don't see anything in Atom 1.0 that should break it. ------------------------------------------------------------------------ Tue, 02 Aug 2005 15:00:50 +0000 : Eaton +1 on that patch. Running it now on my test site and everything looks great. ------------------------------------------------------------------------ Tue, 02 Aug 2005 16:24:54 +0000 : Bèr Kessels a small issue, whe do not use if ($data) $items[$item][$tag] .= $data; please just use if ($data) { $items[$item][$tag] .= $data; } ------------------------------------------------------------------------ Wed, 03 Aug 2005 02:18:54 +0000 : Prometheus6 Attachment: http://drupal.org/files/issues/AGGREGATOR_ATOM_HEAD_1.txt (1.99 KB) Not a problem. Here's the corrections. ------------------------------------------------------------------------ Wed, 03 Aug 2005 16:14:14 +0000 : Dries If this is all it takes to parse Atom feeds (to some extend), I'm all for it. Code looks OK. Set to 'ready to be committed' after some testing. ------------------------------------------------------------------------ Wed, 03 Aug 2005 18:59:20 +0000 : kika anybody up for a testing suite tests? http://intertwingly.net/wiki/pie/FormatTests ------------------------------------------------------------------------ Wed, 03 Aug 2005 20:07:37 +0000 : Prometheus6 That would be for the Atom module.

1 0

[drupal-devel] Spam module 2.0, Looking for beta testers
by Lists 04 Aug '05

04 Aug '05

Hello, I have re-written the spam module, and now have a new version that is available for anyone willing to beta test. In re-writing the module, I have also changed the license from the GPL to the Modified BSD license. Because of this licensing change, the module can't (currently) be distributed from the spam project page [1]. Instead, find it here [2]. New Features: - supports spam expiration (aka auto-deletion of old spam content) - detects the same identical content being posted repeatedly - can display message to users when content is blocked by filters - logging, allows you trace how the spam filter is weighing your content Other improvements: - major code cleanup, moving redundant code into functions - heavily commented - uses one database table for tracking all spam content types - simplified tokenizer - numerous major and minor bug fixes Left to be done (patches welcome): - create upgrade script from old spam module - update README.txt and INSTALL.txt - port spam_surbl.module - update trackback.module patch to support new api Many bugs have been fixed in this new version, but as it is a rewrite it is possible that I've also introduced new bugs. As for upgrading from the old version, right now your best bet is to drop your old tables and start over. The module no longer attempts to collect generic statistics as they weren't remotely accurate anyway. Feedback is welcome. Patches are even more welcome. Thanks, -Jeremy [1] http://drupal.org/project/spam [2] http://kerneltrap.org/jeremy/drupal/spam/

1 0

[drupal-devel] [feature] Put all throttle settings in the same place
by drumm 04 Aug '05

04 Aug '05

Issue status update for http://drupal.org/node/27818 Post a follow up: http://drupal.org/project/comments/add/27818 Project: Drupal Version: cvs Component: throttle.module Category: feature requests Priority: normal Assigned to: drumm Reported by: drumm Updated by: drumm Status: patch (code needs review) Attachment: http://drupal.org/files/issues/throttle-modules_0.patch (9.54 KB) drumm Previous comments: ------------------------------------------------------------------------ Sat, 30 Jul 2005 00:21:16 +0000 : drumm Attachment: http://drupal.org/files/issues/screenshot_12.png (37.86 KB) Screenshot 1/2. The modules admin page. ------------------------------------------------------------------------ Sat, 30 Jul 2005 00:24:10 +0000 : drumm Attachment: http://drupal.org/files/issues/screenshot_13.png (15.81 KB) Screenshot 2/2. The throttle modules page. ------------------------------------------------------------------------ Sat, 30 Jul 2005 00:29:32 +0000 : drumm Attachment: http://drupal.org/files/issues/throttle-modules.patch (9.73 KB) This patch needs http://drupal.org/node/27804. The throttle configuration is currently spread out on the modules page and the block configuration page. This patch moves the module throttling to a local task next to the throttle settings page. I have not worked on moving block throttling yet since I want to see what the reaction to this first part is. If this is comitted I plan to move the block throttle checkboxes and continue improving the modules page. With less interface elements the modules page can be made even simpler that is it now. ------------------------------------------------------------------------ Sat, 30 Jul 2005 19:48:47 +0000 : drumm forgot to label it as a patch ------------------------------------------------------------------------ Mon, 01 Aug 2005 20:21:27 +0000 : factoryjoe +1 for moving throttle/performance settings to it's own page/tab -1 for showing disabled modules; only list those that can be disabled Issues: * "turning on" throttling seems counter intuitive... like, all the modules should be checked as given priority... then you DESELECT in order to take it off the priority list. I guess the current UI is opposite of most conventions... where you start with everything enabled and then take things /off/ the list. * The list is not a scalable UI. If I have 60 modules (*cough cough*) I'll see the first 10-15 and ignore the rest. If you could sort by processing weight ("these modules suck up tonnes of memory! disable these first!") or by functionality, the list would be much more manageable. * I still think "throttle" is a terrible name. Like "taxonomy" people don't readily know what it means and avoid it like the plague. From a marketing perspective, this is bad news for Drupal since it may appears that Drupal is slow when it really just needs to have its *performance* settings tweaked. ------------------------------------------------------------------------ Mon, 01 Aug 2005 20:52:00 +0000 : killes(a)www.drop.org -1 for adding yet another config page. teh throttle setting is fine where it is. ------------------------------------------------------------------------ Wed, 03 Aug 2005 23:05:03 +0000 : drumm Here is an updated patch which has all of the modules checked and may be unchecked to disable them while throttled. It removes all disabled modules and required modules from the list. The page name 'throttle' is generated directly from the module's name. Changing the module name or changing the mechanism which sets that title is out of the scope of this patch. Ordering by processing time or functionality can be done given a decent algorithm to do that. Figuring out such an algorithm would be out of the scope of this patch too since it would need some sort of monitoring functionality as well. I think alphabetical is fine. Does anyone have more feedback on creation of this local task page or keeping it in the modules page?

1 0

[drupal-devel] [feature] Put all throttle settings in the same place
by drumm 04 Aug '05

04 Aug '05

Issue status update for http://drupal.org/node/27818 Post a follow up: http://drupal.org/project/comments/add/27818 Project: Drupal Version: cvs Component: throttle.module Category: feature requests Priority: normal Assigned to: drumm Reported by: drumm Updated by: drumm Status: patch (code needs review) Here is an updated patch which has all of the modules checked and may be unchecked to disable them while throttled. It removes all disabled modules and required modules from the list. The page name 'throttle' is generated directly from the module's name. Changing the module name or changing the mechanism which sets that title is out of the scope of this patch. Ordering by processing time or functionality can be done given a decent algorithm to do that. Figuring out such an algorithm would be out of the scope of this patch too since it would need some sort of monitoring functionality as well. I think alphabetical is fine. Does anyone have more feedback on creation of this local task page or keeping it in the modules page? drumm Previous comments: ------------------------------------------------------------------------ Sat, 30 Jul 2005 00:21:16 +0000 : drumm Attachment: http://drupal.org/files/issues/screenshot_12.png (37.86 KB) Screenshot 1/2. The modules admin page. ------------------------------------------------------------------------ Sat, 30 Jul 2005 00:24:10 +0000 : drumm Attachment: http://drupal.org/files/issues/screenshot_13.png (15.81 KB) Screenshot 2/2. The throttle modules page. ------------------------------------------------------------------------ Sat, 30 Jul 2005 00:29:32 +0000 : drumm Attachment: http://drupal.org/files/issues/throttle-modules.patch (9.73 KB) This patch needs http://drupal.org/node/27804. The throttle configuration is currently spread out on the modules page and the block configuration page. This patch moves the module throttling to a local task next to the throttle settings page. I have not worked on moving block throttling yet since I want to see what the reaction to this first part is. If this is comitted I plan to move the block throttle checkboxes and continue improving the modules page. With less interface elements the modules page can be made even simpler that is it now. ------------------------------------------------------------------------ Sat, 30 Jul 2005 19:48:47 +0000 : drumm forgot to label it as a patch ------------------------------------------------------------------------ Mon, 01 Aug 2005 20:21:27 +0000 : factoryjoe +1 for moving throttle/performance settings to it's own page/tab -1 for showing disabled modules; only list those that can be disabled Issues: * "turning on" throttling seems counter intuitive... like, all the modules should be checked as given priority... then you DESELECT in order to take it off the priority list. I guess the current UI is opposite of most conventions... where you start with everything enabled and then take things /off/ the list. * The list is not a scalable UI. If I have 60 modules (*cough cough*) I'll see the first 10-15 and ignore the rest. If you could sort by processing weight ("these modules suck up tonnes of memory! disable these first!") or by functionality, the list would be much more manageable. * I still think "throttle" is a terrible name. Like "taxonomy" people don't readily know what it means and avoid it like the plague. From a marketing perspective, this is bad news for Drupal since it may appears that Drupal is slow when it really just needs to have its *performance* settings tweaked. ------------------------------------------------------------------------ Mon, 01 Aug 2005 20:52:00 +0000 : killes(a)www.drop.org -1 for adding yet another config page. teh throttle setting is fine where it is.

1 0

[drupal-devel] [task] Remove /view from user links on search
by drumm 04 Aug '05

04 Aug '05

Issue status update for http://drupal.org/node/28154 Post a follow up: http://drupal.org/project/comments/add/28154 Project: Drupal Version: cvs Component: user.module Category: tasks Priority: minor Assigned to: drumm Reported by: drumm Updated by: drumm Status: patch (code needs review) Attachment: http://drupal.org/files/issues/user.module_16.diff (829 bytes) Don't need that /view on the end of the url. drumm

1 0

[drupal-devel] [bug] Statistic information about node reads is shown to unprivileged users
by Robin Monks 03 Aug '05

03 Aug '05

Issue status update for http://drupal.org/node/22565 Post a follow up: http://drupal.org/project/comments/add/22565 Project: Drupal Version: cvs Component: statistics.module Category: bug reports Priority: normal Assigned to: Robin Monks Reported by: massabob Updated by: Robin Monks Status: patch (ready to be committed) /the term "access statistics counter" doesn't tell me anything. The counters are for posts -- I think that should be reflected in the permission's name. / Suggestions: view node access counter view node view counter access node view counter view node hit counter I rather like "view node access counter". Dries, would this satisify you? /Is this different from the 'Count content views' setting?/ Yes, on two counts. * These permissions are for users, not for admins. The count content views setting changes both * This setting lets the administrator choose specific groups to see the counter (eg, moderators, or only registered users) Robin Robin Monks Previous comments: ------------------------------------------------------------------------ Wed, 11 May 2005 07:51:40 +0000 : massabob Information about '%count reads' in node's footer is shown to unprivileged users. I suggest that function statistics_link in statistics.module should be corrected in this way: // Original function statistics_link($type, $node = 0, $main = 0) { global $id; $links = array(); if ($type != 'comment' && variable_get('statistics_display_counter', 0)) { $statistics = statistics_get($node->nid); if ($statistics) { $links[] = format_plural($statistics['totalcount'], '1 read', '%count reads'); } } return $links; } // Fixed function statistics_link($type, $node = 0, $main = 0) { global $id; $links = array(); if ($type != 'comment' && variable_get('statistics_display_counter', 0) && user_access('display statistics')) { $statistics = statistics_get($node->nid); if ($statistics) { $links[] = format_plural($statistics['totalcount'], '1 read', '%count reads'); } } return $links; } The only change is in "&& user_access('display statistics')" on line 98. ------------------------------------------------------------------------ Fri, 27 May 2005 01:44:25 +0000 : RobRoy Should read user_access('access statistics') not user_access('display statistics'). ------------------------------------------------------------------------ Fri, 27 May 2005 11:19:36 +0000 : Robin Monks Attachment: http://drupal.org/files/issues/user.access.stats.patch (730 bytes) And here that is in patch form. Robin ------------------------------------------------------------------------ Wed, 01 Jun 2005 04:09:16 +0000 : Steven I'm not sure about this patch: often, read counts are shown directly on the site. But if the permission for viewing the counts is the same as the permission for accessing the administrator's detailed logs, then you wouldn't give that to everyone. There is already an option to choose whether counts are displayed. Perhaps we could change that to "No" "For priviledged users" "For everyone". In last case it acts like it is now, it the second case it requires "access statistics" permission. What do you think? ------------------------------------------------------------------------ Wed, 01 Jun 2005 14:13:54 +0000 : Robin Monks Sounds good to me. I'll try to code something up for this. Robin ------------------------------------------------------------------------ Thu, 09 Jun 2005 13:55:49 +0000 : Robin Monks Attachment: http://drupal.org/files/issues/authstats.patch (2.26 KB) Here is the patch. Uses a switch to choose between signed in users, all users, users with permissions or noone. Robin ------------------------------------------------------------------------ Thu, 09 Jun 2005 14:08:16 +0000 : Robin Monks I tested this patch with various settings on my local install and it worked fine. Robin ------------------------------------------------------------------------ Thu, 09 Jun 2005 16:30:25 +0000 : Bèr Kessels Is there a reason why you check for $user->uid? Whaen someone has "access statistics" set to anonymous users, your check for $user->uid will override taht settings. Not good IMO. <?php $group .= form_radios(t('Display counter values'), 'statistics_display_counter', variable_get('statistics_display_counter', 0), array('1' => t('For all users'), '2' => t('For authenticated users'), '3' => t('For priviledged users'), '0' => t('Disabled')), t('Display how many times given content has been viewed.')); ?> is very inconsistent. please use *only* the permissions page to set permissions, and do not create new permissions-alike settings in any configuration pages. I would say a simple check for user_access('access statistics') will do the trick ------------------------------------------------------------------------ Thu, 09 Jun 2005 16:32:53 +0000 : Bèr Kessels sorry, i meant to say user_access('access statistics counter'), not user_access('access statistics'). We already have "access statistics'" an additional "access statistics counter" for showing users the counter should work Ber ------------------------------------------------------------------------ Thu, 09 Jun 2005 16:49:48 +0000 : Robin Monks Attachment: http://drupal.org/files/issues/statistics.module (23.1 KB) OK, here is a patch to that end... Robin ------------------------------------------------------------------------ Thu, 09 Jun 2005 17:00:50 +0000 : Robin Monks Attachment: http://drupal.org/files/issues/authstats_0.patch (1 KB) Um, let's just pretend I didn't just upload the entire stats module. OK? OK! Robin ------------------------------------------------------------------------ Thu, 09 Jun 2005 17:05:59 +0000 : Robin Monks Attachment: http://drupal.org/files/issues/authstats_1.patch (1.06 KB) Hotfix. Robin ------------------------------------------------------------------------ Thu, 09 Jun 2005 17:11:23 +0000 : Robin Monks Attachment: http://drupal.org/files/issues/authstats_2.patch (1.11 KB) Hopefuly the final version. Thanks to chx and berkes for pulling it apart ;-) Robin ------------------------------------------------------------------------ Thu, 09 Jun 2005 17:21:38 +0000 : Bèr Kessels +1 for this patch. it adds functionality, but does not add clutter nor any config options. ------------------------------------------------------------------------ Fri, 10 Jun 2005 02:39:06 +0000 : frjo +1 I have implemented the patch authstats_2.patch on my site running Drupal 4.6.1 and it works well. ------------------------------------------------------------------------ Fri, 01 Jul 2005 08:10:22 +0000 : mfb +1 ------------------------------------------------------------------------ Fri, 01 Jul 2005 12:03:56 +0000 : Jose A Reyero +1 I tested the patch, works fine, and I'm always for "more options" ;-) ------------------------------------------------------------------------ Mon, 01 Aug 2005 21:17:48 +0000 : Junyor +1. Using this on my site now. ------------------------------------------------------------------------ Tue, 02 Aug 2005 21:53:15 +0000 : Robin Monks Attachment: http://drupal.org/files/issues/node_counter_permission.patch (2.97 KB) This patch revision removes the duplicate show counter option from the settings. Thanks UnConeD! Robin ------------------------------------------------------------------------ Wed, 03 Aug 2005 15:44:59 +0000 : Dries -1, the term "access statistics counter" doesn't tell me anything. The counters are for posts -- I think that should be reflected in the permission's name. Is this different from the 'Count content views' setting?

1 0

[drupal-devel] [feature] Trigger debug mode on-the-fly
by mathias 03 Aug '05

03 Aug '05

Issue status update for http://drupal.org/node/21379 Project: Drupal Version: cvs Component: other Category: feature requests Priority: normal Assigned to: Anonymous Reported by: mathias Updated by: mathias Status: patch Attachment: http://drupal.org/files/issues/devel_GET_params_core.patch (2.38 KB) This patch allows admins who are using devel.module to optionally toggle the page timer, query logger and page redirection for each request using GET parameters rather than navigating to admin/settings/devel and flipping the global settings back and forth. I added this since I usually don't need to print out this information for every page. Now I just use: http://example.com/index.php?q=node&dev_query=1 for the request I'm interested in. Two other params can also be toggled: dev_timer and dev_redirect. mathias

4 5