February 2005 - Development - lists.drupal.org

[drupal-devel] [feature] Add support for generating an RSS 0.92 feed of news aggregator items
by Boris Mann 09 Feb '05

09 Feb '05

Project: Drupal Version: cvs Component: aggregator.module Category: feature requests Priority: normal Assigned to: Anonymous Reported by: blake7 Updated by: Boris Mann Status: patch Here's another bump to this. I would say since it missed 4.5 (and was ready at the time) we get this in to 4.6. Should mesh nicely with walkah's RSS updates. Dries: there is an XML icon at the bottom of each category page, plus the HTML header can include an alternate link to the feed. I vote against any sort of "rss.xml", people can alias if they wish. (ideally we could get per-category OPML links as well, but I'll leave that for another feature request) Boris Mann Previous comments: ------------------------------------------------------------------------ July 8, 2004 - 20:11 : blake7 Attachment: http://drupal.org/files/issues/aggregator.module_0.patch (2.96 KB) The attached patch adds support for generating an RSS 0.92 feed of the default news aggregator page (the output of aggregator_page_last()). This allows re-use of the Drupal aggregator in other RSS readers like Trillian. The patch consists of the aggregator_page_rss() function which builds the XML output needed. It uses the existing RSS building functions already present to construct the feed. Note the comment about how many items are in the feed. In Drupal 4.4.x, there was a variable "aggregator_page_limit" which represented how many items were returned for both the HTML and RSS versions. It defaulted to 75 items. In Drupal CVS, all the HTML output is now paged through a hard-coded limit of 20 items per page. A limit of 20 seems small for the RSS feed, so I reverted to the old limit of 75. Would it be better to add a new variable like "aggregator_rss_limit"? Also, there are 2 places that the callback to aggregator_page_rss are installed. One is in _aggregator_page_list right before the pager output and shows up as the XML icon (like how the OPML feeds are linked now). The other is in aggregator_menu where I try and install a menu itme called "RSS feed". For some reason, the menu itme is not displaying itself though I believe I am inserting it properly. Feel free to contact me if there are any questions. Thanks! ------------------------------------------------------------------------ July 9, 2004 - 09:29 : blake7 Attachment: http://drupal.org/files/issues/aggregator.module_1.patch (2.54 KB) Fixed the navigation menu problems mentioned above. Please consider this patch instead of the first one. ------------------------------------------------------------------------ July 9, 2004 - 10:09 : drumm When this functionality is added it should be able to generate feeds for each aggregator category as well. ------------------------------------------------------------------------ July 9, 2004 - 17:22 : blake7 Attachment: http://drupal.org/files/issues/aggregator.module_2.patch (3.25 KB) Done. The attached patch supports RSS feeds of each aggregator category through the XML icon at the bottom exactly like the main aggregator page. ------------------------------------------------------------------------ July 10, 2004 - 04:01 : Dries 1. $base_url . "/aggregator" . $url is incorrect. All URLs must be constructed using url() as it is aware of clean URLs and path aliasing. 2. Emitting 75 RSS items is quite much (uncommon) and can easily turn into a bandwidth bottleneck. Can we add one global setting that controls how many entries our feed should have (not just the newly introduced feeds)? To 'export' your content in case you wish to migrate to another tool, a special value 'all' (say '1000000') might be handy. ------------------------------------------------------------------------ September 17, 2004 - 00:19 : blake7 Attachment: http://drupal.org/files/issues/aggregator.module_3.patch (4.47 KB) 1. Using url() now. 2. Added the presistent variable aggregator_rss_limit which controls how many RSS items are generated. A form_select edit control to control the new variable, with all defined as 1000000, was added to the configure tab. I am not sure what you mean by "our feeds" versus "newly introduced feeds" in your comment. Also, even though the code-freeze has taken place, do you think it would be possible to get this into Drupal 4.5? The code change is very minimal and I believe the ability to share aggregated feeds is a great feature. ------------------------------------------------------------------------ September 17, 2004 - 10:07 : Boris Mann +1 I actually assumed (there was much discussion?) that the re-exporting of aggregator items on a per-bundle basis was going to be enabled for 4.5. Let's get this in before the code freeze -- this would be a major feature announcement for 4.5, and could perform the basis for other great functionality for 4.6 (e.g. per category OPML file generation). ------------------------------------------------------------------------ October 22, 2004 - 10:20 : blake7 Now that 4.5 has released. Can this go into CVS/HEAD? Is there any other changes to the patch that need to be made by me to make that happen? Thanks. ------------------------------------------------------------------------ October 23, 2004 - 10:05 : Dries I'll look at it again. Does the patch add XML-icons or how do people find out about the RSS feeds? Can't we make the URLs a bit more intuitive? For example by making it so that you merely have to append '/rss.xml' to the aggregator module's URLs: foo.com/aggregator/rss.xml foo.com/aggregator/categories/1/rss.xml It is certainly not a requirement, just a thought that might be worth considering. ------------------------------------------------------------------------ October 23, 2004 - 10:15 : Dries I'll look at it again. Does the patch add XML-icons or how do people find out about the RSS feeds? Can't we make the URLs a bit more intuitive? For example by making it so that you merely have to append '/rss.xml' to the aggregator module's URLs: foo.com/aggregator/rss.xml foo.com/aggregator/categories/1/rss.xml It is certainly not a requirement, just a thought that might be worth considering. ------------------------------------------------------------------------ October 25, 2004 - 02:50 : Bèr Kessels cross linking to a [feature] that talks about such a unified rss link: http://drupal.org/node/11937 ------------------------------------------------------------------------ February 9, 2005 - 12:17 : sethcohn Bump. Neat feature, would be nice for 4.6 What's the progress on making it happen? -- View: http://drupal.org/node/9167 Edit: http://drupal.org/project/comments/add/9167

1 0

[drupal-devel] [feature] Add support for generating an RSS 0.92 feed of news aggregator items
by sethcohn 09 Feb '05

09 Feb '05

Project: Drupal Version: cvs Component: aggregator.module Category: feature requests Priority: normal Assigned to: Anonymous Reported by: blake7 Updated by: sethcohn Status: patch Bump. Neat feature, would be nice for 4.6 What's the progress on making it happen? sethcohn Previous comments: ------------------------------------------------------------------------ July 9, 2004 - 04:11 : blake7 Attachment: http://drupal.org/files/issues/aggregator.module_0.patch (2.96 KB) The attached patch adds support for generating an RSS 0.92 feed of the default news aggregator page (the output of aggregator_page_last()). This allows re-use of the Drupal aggregator in other RSS readers like Trillian. The patch consists of the aggregator_page_rss() function which builds the XML output needed. It uses the existing RSS building functions already present to construct the feed. Note the comment about how many items are in the feed. In Drupal 4.4.x, there was a variable "aggregator_page_limit" which represented how many items were returned for both the HTML and RSS versions. It defaulted to 75 items. In Drupal CVS, all the HTML output is now paged through a hard-coded limit of 20 items per page. A limit of 20 seems small for the RSS feed, so I reverted to the old limit of 75. Would it be better to add a new variable like "aggregator_rss_limit"? Also, there are 2 places that the callback to aggregator_page_rss are installed. One is in _aggregator_page_list right before the pager output and shows up as the XML icon (like how the OPML feeds are linked now). The other is in aggregator_menu where I try and install a menu itme called "RSS feed". For some reason, the menu itme is not displaying itself though I believe I am inserting it properly. Feel free to contact me if there are any questions. Thanks! ------------------------------------------------------------------------ July 9, 2004 - 17:29 : blake7 Attachment: http://drupal.org/files/issues/aggregator.module_1.patch (2.54 KB) Fixed the navigation menu problems mentioned above. Please consider this patch instead of the first one. ------------------------------------------------------------------------ July 9, 2004 - 18:09 : drumm When this functionality is added it should be able to generate feeds for each aggregator category as well. ------------------------------------------------------------------------ July 10, 2004 - 01:22 : blake7 Attachment: http://drupal.org/files/issues/aggregator.module_2.patch (3.25 KB) Done. The attached patch supports RSS feeds of each aggregator category through the XML icon at the bottom exactly like the main aggregator page. ------------------------------------------------------------------------ July 10, 2004 - 12:01 : Dries 1. $base_url . "/aggregator" . $url is incorrect. All URLs must be constructed using url() as it is aware of clean URLs and path aliasing. 2. Emitting 75 RSS items is quite much (uncommon) and can easily turn into a bandwidth bottleneck. Can we add one global setting that controls how many entries our feed should have (not just the newly introduced feeds)? To 'export' your content in case you wish to migrate to another tool, a special value 'all' (say '1000000') might be handy. ------------------------------------------------------------------------ September 17, 2004 - 08:19 : blake7 Attachment: http://drupal.org/files/issues/aggregator.module_3.patch (4.47 KB) 1. Using url() now. 2. Added the presistent variable aggregator_rss_limit which controls how many RSS items are generated. A form_select edit control to control the new variable, with all defined as 1000000, was added to the configure tab. I am not sure what you mean by "our feeds" versus "newly introduced feeds" in your comment. Also, even though the code-freeze has taken place, do you think it would be possible to get this into Drupal 4.5? The code change is very minimal and I believe the ability to share aggregated feeds is a great feature. ------------------------------------------------------------------------ September 17, 2004 - 18:07 : Boris Mann +1 I actually assumed (there was much discussion?) that the re-exporting of aggregator items on a per-bundle basis was going to be enabled for 4.5. Let's get this in before the code freeze -- this would be a major feature announcement for 4.5, and could perform the basis for other great functionality for 4.6 (e.g. per category OPML file generation). ------------------------------------------------------------------------ October 22, 2004 - 18:20 : blake7 Now that 4.5 has released. Can this go into CVS/HEAD? Is there any other changes to the patch that need to be made by me to make that happen? Thanks. ------------------------------------------------------------------------ October 23, 2004 - 18:05 : Dries I'll look at it again. Does the patch add XML-icons or how do people find out about the RSS feeds? Can't we make the URLs a bit more intuitive? For example by making it so that you merely have to append '/rss.xml' to the aggregator module's URLs: foo.com/aggregator/rss.xml foo.com/aggregator/categories/1/rss.xml It is certainly not a requirement, just a thought that might be worth considering. ------------------------------------------------------------------------ October 23, 2004 - 18:15 : Dries I'll look at it again. Does the patch add XML-icons or how do people find out about the RSS feeds? Can't we make the URLs a bit more intuitive? For example by making it so that you merely have to append '/rss.xml' to the aggregator module's URLs: foo.com/aggregator/rss.xml foo.com/aggregator/categories/1/rss.xml It is certainly not a requirement, just a thought that might be worth considering. ------------------------------------------------------------------------ October 25, 2004 - 10:50 : Bèr Kessels cross linking to a [feature] that talks about such a unified rss link: http://drupal.org/node/11937 -- View: http://drupal.org/node/9167 Edit: http://drupal.org/project/comments/add/9167

1 0

[drupal-devel] Release critical bugs for February 09, 2005
by drupal-devel＠drupal.org 09 Feb '05

09 Feb '05

Adc / support requests db corruption? age: 3 days 8 hours url: http://drupal.org/node/16869 Bbcode / bug reports email spam filtering does not "see" email addresses when they are NOT preceded by a word age: 3 weeks 4 days url: http://drupal.org/node/15654 Chatbox / support requests Cron run kills Chat? age: 9 weeks 1 day url: http://drupal.org/node/14032 Drupal / bug reports node preview broken -- no clone() function age: 1 day 21 hours url: http://drupal.org/node/16958 "TYPES" is not a legal field name table "BLOCKS" age: 4 days 5 hours url: http://drupal.org/node/16817 Published poll doesnt show up in poll block if in submission queue age: 6 days 15 hours url: http://drupal.org/node/16671 Adding two custom blocks with the same name produces a sql warning age: 6 days 18 hours url: http://drupal.org/node/16659 Some menu items become fixed or undeletable age: 1 week 1 day url: http://drupal.org/node/16516 Cache-Control header not set age: 1 week 6 days url: http://drupal.org/node/16295 Drupal adds extra items to body tag, doesn't line break output html after move to new server age: 2 weeks 4 days url: http://drupal.org/node/16004 updates.inc deleting profile_data when updating age: 3 weeks 3 days url: http://drupal.org/node/15686 Change to sesssion.inc violates UID database constraint age: 3 weeks 3 days url: http://drupal.org/node/15666 problem with forum.module with postgresql v7.4.1 age: 4 weeks 2 days url: http://drupal.org/node/15412 forum.module problem with postgresql v7.4.1 age: 4 weeks 2 days url: http://drupal.org/node/15411 postgresql error age: 4 weeks 6 days url: http://drupal.org/node/15175 UID 0, anonymous user account deleted age: 9 weeks 3 days url: http://drupal.org/node/13915 Styles broken after multisite patch assigned: TDobes age: 10 weeks 10 hours url: http://drupal.org/node/13738 Entering an empty title broke drupal age: 12 weeks 2 days url: http://drupal.org/node/12974 Argument NOT must be type boolean age: 12 weeks 2 days url: http://drupal.org/node/12950 Comments in approval queue still show up in tracker age: 16 weeks 4 days url: http://drupal.org/node/11647 Drupal / feature requests Don't reset existing password on request, prevent DoS password reset abuse age: 2 days 17 hours url: http://drupal.org/node/16909 User info exposed to anonymous user age: 10 weeks 4 days url: http://drupal.org/node/13545 Allow named anchors to work without specifying full path to node age: 1 year 11 weeks url: http://drupal.org/node/4213 Drupal / support requests Problem with Forums age: 4 weeks 4 days url: http://drupal.org/node/15299 /?q=admin/modules doesnt work for me! age: 11 weeks 4 days url: http://drupal.org/node/13193 Drupal / tasks atom.module working version to 4.5.0 assigned: hugoromano age: 14 weeks 4 hours url: http://drupal.org/node/12496 Postgres Compatibility Update assigned: adrian age: 20 weeks 2 days url: http://drupal.org/node/10945 file.inc code review assigned: Stefan age: 42 weeks 2 days url: http://drupal.org/node/7235 Drupal.org maintenance / bug reports Konqueror problem age: 2 days 12 hours url: http://drupal.org/node/16913 screenshots : page not found age: 5 days 4 hours url: http://drupal.org/node/16752 Large Fonts problems age: 6 days 17 hours url: http://drupal.org/node/16664 I can not add a release of Chinese Simplified Translation. age: 16 weeks 8 hours url: http://drupal.org/node/11770 Drupal.org maintenance / support requests CVS error age: 1 week 2 days url: http://drupal.org/node/16475 Event / bug reports search menu tabs prints "array" age: 4 days 21 hours url: http://drupal.org/node/16788 search over fields.inc fields conceptual error age: 14 weeks 5 days url: http://drupal.org/node/12259 fields.inc screws up with multible multiselect fields age: 14 weeks 5 days url: http://drupal.org/node/12257 Still GMT/TZ issues with CVS age: 15 weeks 6 days url: http://drupal.org/node/11870 Time entry is GMT even when site default is not age: 16 weeks 4 days url: http://drupal.org/node/11630 Start Time of Event is NOT Preserved age: 33 weeks 6 days url: http://drupal.org/node/8565 Feature / bug reports Menu item not showing age: 18 weeks 5 days url: http://drupal.org/node/11210 File / bug reports segmentation fault related to filestore age: 1 year 19 weeks url: http://drupal.org/node/3001 Filestore / support requests File store module - Overwriting Filename....Help needed assigned: harishkm age: 45 weeks 9 hours url: http://drupal.org/node/6827 Unserialize error at line 104 age: 51 weeks 5 days url: http://drupal.org/node/5857 Filestore module- sql errors rectified assigned: harishkm age: 1 year 6 days url: http://drupal.org/node/5619 Problems in Filestore2 module assigned: harishkm age: 1 year 1 week url: http://drupal.org/node/5458 Filestore Module Problems unaddressed assigned: harishkm age: 1 year 2 weeks url: http://drupal.org/node/5365 Goofy / bug reports Blocks with no name age: 3 days 13 hours url: http://drupal.org/node/16860 Site Name age: 3 days 13 hours url: http://drupal.org/node/16859 Goofy / support requests database corruption? age: 3 days 8 hours url: http://drupal.org/node/16866 Groups / bug reports Drupal-4.4-rc breaks registration setups and groups.module age: 47 weeks 5 days url: http://drupal.org/node/6392 Htmlarea / bug reports HTMLarea 4.5.0 doesn't work with themes using xtemplate.tmpl age: 2 weeks 1 day url: http://drupal.org/node/16155 Image / bug reports Image.Module does not know the correct filename age: 12 weeks 1 day url: http://drupal.org/node/13032 SAFE MODE Restriction in effect... age: 15 weeks 6 days url: http://drupal.org/node/11843 Uploading jpeg makes Imagemagick do high compression age: 17 weeks 3 days url: http://drupal.org/node/11445 Image module failing on Preview because of a blank node id age: 19 weeks 4 days url: http://drupal.org/node/11069 Similar problem with Directory Upload Slow age: 19 weeks 4 days url: http://drupal.org/node/11064 Interlaced / support requests db corrupt? age: 3 days 8 hours url: http://drupal.org/node/16872 Listhandler / bug reports Security bug: check mail -> assume ID of poster age: 3 days 16 hours url: http://drupal.org/node/16849 Node Aggregator / bug reports Import doesn't like XML/RSS feeds with no Title element age: 36 weeks 4 days url: http://drupal.org/node/8128 Node Aggregator / feature requests RSS items overwriting edited items age: 34 weeks 4 days url: http://drupal.org/node/8455 Notify / feature requests Notify.module ignores the true submission queue age: 1 year 15 weeks url: http://drupal.org/node/3765 Pdfview / feature requests Internationalization support for PDF assigned: TheLibrarian age: 45 weeks 1 day url: http://drupal.org/node/6795 Project / bug reports blank page after install age: 1 week 22 hours url: http://drupal.org/node/16575 "Page Not Found" when trying to view issues or add new release age: 9 weeks 6 days url: http://drupal.org/node/13815 Project pages display only title and description age: 11 weeks 6 days url: http://drupal.org/node/13082 Fatal error: Cannot use string offset as an array in includes/tablesort.inc on line 113 assigned: publian age: 18 weeks 6 days url: http://drupal.org/node/11187 Can't create new project with issue tracker age: 25 weeks 13 hours url: http://drupal.org/node/10148 Project / support requests Help with everything - nothing seems to work in the latest build age: 11 weeks 2 days url: http://drupal.org/node/13310 Recipe / bug reports Sql error when sitename/recipe is visited age: 1 day 19 hours url: http://drupal.org/node/16964 search module menu tabs prints "array" age: 4 days 21 hours url: http://drupal.org/node/16783 Sunflower / support requests database corruption? age: 3 days 8 hours url: http://drupal.org/node/16867 TrackBack / bug reports Sends trackbacks for unpublished nodes age: 2 weeks 1 min url: http://drupal.org/node/16239 Overly aggressive age: 9 weeks 5 days url: http://drupal.org/node/13865 trackback borks google and every other search engine age: 14 weeks 2 days url: http://drupal.org/node/12388

1 0

[drupal-devel] CiviCRM: We need your Input by 2/14/2005
by David Geilhufe 09 Feb '05

09 Feb '05

[apologies for somewhat excessive cross posting] CiviCRM is seeking input on the design documents for our open-source (GPL) relationship-management application for the NPO/NGO sector. Four full-time developers will start coding CiviCRM to the design documents on 2/14/05. Get your input in by then!! ~~~~~~~~~~~~~~~~~ WHAT IS CiviCRM? ~~~~~~~~~~~~~~~~~ CiviCRM is a LAMP-based, open-source project to create relationship management software for the nonprofit and nongovernmental sectors. CiviCRM stores information on the universe of people associated with a nonprofit organization and on their interactions (emails, donations, petitions, events, etc.). http://objectledge.org/confluence/display/CRM/CRM+Project+One+Pager ~~~~~~~~~~~~~~~~~ INPUT ON WHAT? ~~~~~~~~~~~~~~~~~ CiviCRM manages contacts and relationships for other software applications. Initially CiviCRM will manage contacts and relationships for the Drupal/ CivicSpace platform (www.civicspacelabs.org) For other software developers to use CiviCRM, they need to know the data model and the public API. The data model defines what is available (individuals, households, organizations, relationships, groups, actions) and the API defines how developers access and manipulate that data. To review documents, follow the hyper link, scroll to the bottom of the page, and click the �add a comment� hyper link. Feature overview (high level): http://objectledge.org/confluence/pages/viewpage.action?pageId=592 Data model: http://objectledge.org/confluence/display/CRM/Data+Model Public API: http://objectledge.org/confluence/display/CRM/CRM+v1.0+Public+APIs ~~~~~~~~~~~~~~~~~~~~~~~~~~~ HEY, I�M NOT A CODE-HEAD! ~~~~~~~~~~~~~~~~~~~~~~~~~~~ If the documents above make your head hurt, you can still make a valuable contribution! Write a user narrative for us about how a NPO/NGO would store, group and manipulate information about individuals, households and organizations. User narratives are plain English stories about what a NPO/NGO would do with CiviCRM. To contribute a user narrative, follow the link below, read the examples, scroll to the bottom and click the �add a comment� hyper link. http://objectledge.org/confluence/display/CRM/User+Narratives ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ THIS IS COOL, HOW CAN I HELP IN MY COPIOUS SPARE TIME? ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ CiviCRM is an open, community process. We encourage broad participation. The more you contribute time and effort, the more impact you will have on CiviCRM. Find out how to participate here: http://objectledge.org/confluence/display/CRM/How+to+Participate Alternatively, drop me, David Geilhufe, an email at dgeilhufe AT-yahoo DOT-com. We�ll find a good way for you to contribute and get all your questions answered. Please distribute this announcement to anyone that you think needs to know about CiviCRM David Geilhufe CiviCRM Team Member dgeilhufe AT-yahoo DOT-com __________________________________________________ Do You Yahoo!? Tired of spam? Yahoo! Mail has the best spam protection around http://mail.yahoo.com

1 0

[drupal-devel] Trailing slash problem
by Chris Johnson 09 Feb '05

09 Feb '05

I'm developing a module to search a newspaper index database. All is proceeding fairly well, and the functionality I've implemented so far works -- except for one small oddity that I am having trouble chasing down. Has anyone seen anything like the following before? If I go to my search form, e.g. http://localhost/newspaper, and fill in some search keys and hit submit, I get the same page back, and the form filled in, but no results. However, the URL is changed to have a trailing "/", that is: http://localhost/newspaper/. If I just hit submit again, the search works perfectly and returns results. In other words, the search does not work unless the trailing "/" is on the URL, and I can't tell why, or what is appending it there. I'm sure it's a simple typo someplace. I've combed through the code a few times without success. My module itself does not alter the URL, so it's a side effect of probably something in common.inc or menu.inc, I'm thinking. Suggestions? -- Chris Johnson

1 0

[drupal-devel] Link broken Screenshots on main page
by Steven Peck 09 Feb '05

09 Feb '05

BuddaBoy mentioned that the screenshot link on the main page was broken http://drupal.org/image goes to Page not found.

2 1

[drupal-devel] [bug] Usability inconsistancy: blog should not tell user he cannot add blogs.
by Bèr Kessels 08 Feb '05

08 Feb '05

Project: Drupal Version: cvs Component: blog.module Category: bug reports Priority: normal Assigned to: Anonymous Reported by: Bèr Kessels Updated by: Bèr Kessels Status: patch Attachment: http://drupal.org/files/issues/blog_remove_incosistant_perm.patch (668 bytes) In user pages we now tell users thye cannot post blogs if they have not got proper permissions. This is incosistant with other modules (if you have not got permissions for adding recipies it does not tell you so in your profile, and it should not.) Thus i suggest we simply remove it. See attached patch. Bèr Kessels -- View: http://drupal.org/node/17024 Edit: http://drupal.org/project/comments/add/17024

2 1

[drupal-devel] [task] Remove page module help text.
by Dries 08 Feb '05

08 Feb '05

Project: Drupal Version: cvs Component: page.module Category: tasks Priority: normal Assigned to: drumm Reported by: drumm Updated by: Dries Status: patch Committed to HEAD. Dries Previous comments: ------------------------------------------------------------------------ February 8, 2005 - 01:10 : drumm Attachment: http://drupal.org/files/issues/page.module_2.diff (1.56 KB) The page module's long help text is a bunch of lies and then it briefly explains it's permissions. IMO it should just be taken out. I can't think of what help should be there. ------------------------------------------------------------------------ February 8, 2005 - 01:41 : drumm Status should be patch. -- View: http://drupal.org/node/16962 Edit: http://drupal.org/project/comments/add/16962

1 0

[drupal-devel] [bug] Error while deleting user
by Dries 08 Feb '05

08 Feb '05

Project: Drupal Version: cvs Component: user.module Category: bug reports Priority: normal Assigned to: chx Reported by: chx Updated by: Dries Status: patch buddy: can you reproduce this problem with CVS HEAD or with Drupal 4.5.x? Dries Previous comments: ------------------------------------------------------------------------ January 12, 2005 - 21:44 : chx Attachment: http://drupal.org/files/issues/user_delete_0.patch (875 bytes) Type error Date Wednesday, January 12, 2005 - 19:17 User chx Location /cvs/drupal/?q=user/2/edit Message array_keys(): The first argument should be an array in /var/www/cvs/drupal/modules/user.module on line 1048. Hostname 127.0.0.1 ------------------------------------------------------------------------ January 25, 2005 - 00:20 : Dries I can't reproduce this problem. ------------------------------------------------------------------------ February 7, 2005 - 20:00 : budda Problem is fixed by the patch here. Had no more problems since. Error was re-produced when trying to delete the same user account a number of times by hitting the 'delete' form button. ------------------------------------------------------------------------ February 7, 2005 - 20:20 : budda I also note that you can hit the 'delete' button as much as you want and the account will not be deleted. I see that you are required to first mark the account status to "block". Actually giving user feedback to say this would be handy. Currently clicking the 'delete' button just refreshes the user form. Maybe some javascript could be used to disable the 'delete' button until the status == 0 (blocked) ? Definitely increase usability then. Is this a related bug or a new issue? -- View: http://drupal.org/node/15511 Edit: http://drupal.org/project/comments/add/15511

1 0

[drupal-devel] [bug] QUICKFIX: broken db_query
by killes 08 Feb '05

08 Feb '05

Project: Drupal Version: cvs Component: upload.module Category: bug reports Priority: normal Assigned to: killes(a)www.drop.org Reported by: killes(a)www.drop.org Updated by: killes(a)www.drop.org Status: patch Attachment: http://drupal.org/files/issues/upload_2.patch (754 bytes) The db_rewrite_sql patch created a broken query. killes(a)www.drop.org -- View: http://drupal.org/node/16993 Edit: http://drupal.org/project/comments/add/16993

1 0