Closed (fixed)
Project:
Drupal core
Version:
4.6.3
Component:
blog.module
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
8 Feb 2005 at 23:27 UTC
Updated:
29 Nov 2006 at 17:41 UTC
Jump to comment: Most recent file
Small patch to show a list of bloggers. We all should know that blog.module is not meant for personal blogging, but that set aside, we do not have a way to show all bloggers, on a site as a list. This patch ads a few lines that will allow you to show that overview of blogging users. It ads a menu item, disabled by default that shows such a list.
| Comment | File | Size | Author |
|---|---|---|---|
| #21 | blog_bloggers_list_1.patch | 1.75 KB | Bèr Kessels |
| #18 | blog_bloggers_list_0.patch | 1.75 KB | igrcic |
| #8 | blog_bloggers_list_2_0.patch | 1.71 KB | Bèr Kessels |
| #7 | blog_bloggers_list_2.patch | 1.81 KB | Bèr Kessels |
| #2 | list_bloggers_2.png | 2.25 KB | Bèr Kessels |
Comments
Comment #1
Bèr Kessels commentedHere is a small screeny that shows what this is about.
Comment #2
Bèr Kessels commentedhmm. error in my png file. Here is a correct one.
Comment #3
Emiliano commentedHi Bèr Kessels!
Thanks for the patch! It's really useful! :-)
I just noticed what I think is a little mistake in this line:
$items[] = l($row->name, 'blog/'.$row->uid)." ".theme('xml_icon', url('blog/'.$row->uid.'/feed'), TRUE);
Which I believe should end like:
$items[] = l($row->name, 'blog/'.$row->uid)." ".theme('xml_icon', url('blog/feed/'.$row->uid), TRUE);
Ah, and BTW, "db_rewrite_sql" didn't work for me... I just cut it off... ;-)
Cheers,
Emiliano.
Comment #4
(not verified) commentedComment #5
Bèr Kessels commentedComment #6
Bèr Kessels commentedComment #7
Bèr Kessels commentedand here is the actual patch. (i think i really need some script that tells me i did not attach a patch ;) )
Comment #8
Bèr Kessels commentedsigh. something tells me i should stop working.... a silly "F" sneaked into the previous patch.
Comment #9
(not verified) commentedI applied the patch, reuploaded the blog.module and then deactivated and reactivated it. I cannot see a difference. How do I access the blog users page?
Drupal 4.6rc
thanks for the help.
Comment #10
Bèr Kessels commentedlook in the menu admin.
Comment #11
Bèr Kessels commentedComment #12
Bèr Kessels commentedNearly two months since the patch. If there is no need for this I will mark it won't fix.
Comment #13
tostinni commentedI made a custom block for this, but it still need a little improvment (add "read more" for example). So I don't know if it's usefull but if anyone care :
Comment #14
Chris Johnson commentedI think this is a useful feature. I've coded custom blocks in the past to provide it on my sites.
Comment #15
Andrzej7 commentedI was a little afraid (poor PHP) to install the patch (March...), but I have made a block. And it's very nice.
One question only. All the bloggers are shown, but the Admin (user No 1) is on the list and he has no blogs.
What's that?
Comment #16
Bèr Kessels commentedTo avoid confusion: this patch ads no blocks, but shows the list inline: as content!
Comment #17
igrcic commentedaaargh.....there is a problem when using PostgreSQL db. After I successfully patch blog.module and enable it in menu, click on it and following sql errors pop up:
For first one its clear, there is no such fucntion in Postgresql, and second one is typical Postgresql/Mysql error, n.created isn't in select syntax. But if we put it in a select distinct syntax , it isn't DISTINCT anymore, so I get lots of lines with user and all of his blogs, with different time of course. So we must modify query a bit:
It works fine like this :)
Didn't look for first one yet... will when I catch some time.
Cheers!
Comment #18
igrcic commentedSorry, here is a patch for a blog_bloggers_list_2_0.patch :)
Comment #19
Bèr Kessels commentedThat doesz not seem a patch of a patch, but a regular patch.
Igrirc, please note that the SQL contains unbracketed tablenames. That is unacceptible. If you don't want to fix it, please report it back here, So I can fix it otherwise :)
Comment #20
Bèr Kessels commentedComment #21
Bèr Kessels commentedBingo! My own patch, brought to you by patchbingo!
Comment #22
essig commentedI think 'DISTINCT' can be removed from the query in blog_bloggers_list_1.patch. There is already a 'GROUP BY' clause over the same columns.
George Essig
Comment #23
Bèr Kessels commentedThanks George, you are correct. I changed the query, it is now smaller and faster.
Comment #24
Bèr Kessels commentedI decided to take this out of the queue, for it is far too much work to get something into core. This will be placed insode userposts.module.
Comment #25
Bèr Kessels commentedwhat is with those kind of issues that add consistency and usability. Those kind of issues that are made after often heard requests in the forulms and so?
Why did Dries or Steven not even spend time commenting on it. Even closing this issue would suffice.
This is one of these issues that someone (in this case me) spent a lot of time on, that all people are positive about, yet linger and linger and linger.
Is this another five hors (or so) of my life wasted? No really, this gets frustrating. Every day I stumble upon my issues tha go back ages yet are not closed nor applied. frustrating and demoralizing.
Comment #26
bonobo commentedI had just come onto the site to search for precisely this functionality -- this has some GREAT applications for a blog based class site -- I will be applying the patch to the blog module, and using the custom block created above -- I would love to see both a patch for 4.6.3, and this feature in core for 4.7
If you want patches on this tested, I'm happy to do it.
Thanks for your work on this.
bonobo
Comment #27
bonobo commentedClosing due to age of issue and the existence of the blogger module,