Drupal 6

mradcliffe - February 10, 2009 - 14:47
Project:Blog List
Version:5.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs review
Description

I needed a simple module like this so I ported it to drupal 6.

I also needed a block output so I abstracted the query and sent everything through theme_table as well (to conform with drupal standards).

both the page and block uses the same output.

AttachmentSize
blog_list.patch3.46 KB

#1

heartsutra - February 11, 2009 - 00:01

Thank you!! My timing seems to be very good, because today was the first time I looked at this module, and suddenly there was a brand-new D6 patch ready and waiting.

I had a little trouble applying the patch though... it worked fine on blog_list.module but it failed on blog_list.info, so I made those changes by hand. Here's what happened:

$ patch < blog_list.patch
patching file blog_list.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file blog_list.info.rej
patching file blog_list.module

But broadly speaking, it works! I'll tell the Drupal Multi-User Blogging Group about this :-)

Best wishes,
Andrea

PS: Don't let me stop anyone from reviewing the code... I'd do it myself if I knew how.

#2

heartsutra - February 11, 2009 - 04:44

Hmm, now this is odd.

I have a page view called "Latest Blog Entries", which shows teasers of the ten most recent blog posts, and it uses a pager at the bottom. The Blog List block is in the sidebar of that page.

When I added more than ten blog entries, the pager showed up at the bottom of the Latest Blog Entries page, as I expected, but it also shows up at the bottom of the Blog List block! And when I go to mysite.com/blog to see the Blog List page, the pager is there as well. I only have four blogs listed, and when I click on the "Next" button I get an empty list which only has the "User" heading on top.

#3

mradcliffe - February 11, 2009 - 03:29

That's my fault. I added the pager to the block thinking it would work, and that's not possible in drupal. :)

blog_list-patched.patch patches you from the previous patch to the fix.
blog_list_cvs.patch is a full patch from blog_list cvs.

AttachmentSize
blog_list_cvs.patch 3.75 KB
blog_list-patched.patch 1.69 KB

#4

tsi - April 20, 2009 - 19:21

Will someone please publish the pached files ?
Not so good at patch work :-(

Thanks.

#5

fpatten - April 24, 2009 - 23:43

I Second this. I tried with the instructions posted here http://drupal.org/node/60818 but kept getting the fail error.

patch < blog_list_cvs.patch
patching file blog_list.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file blog_list.info.rej
patching file blog_list.module

The module will not load in Drupal at all.

#6

mradcliffe - April 25, 2009 - 03:10

I just did the following:

cvs co contributions/modules/blog_list
cd contributions/modules/blog_list
wget http://drupal.org/files/issues/blog_list_cvs.patch
patch
-p0 < blog_list_cvs.patch
patching file blog_list.info
patching file blog_list.module

No problems. Attached.

AttachmentSize
blog_list-6.x-dev.tar_.gz 1.67 KB

#7

tsi - April 25, 2009 - 17:22

Thanks ! works for me.

#8

fpatten - April 26, 2009 - 00:26

Tried this version. The module was loaded but I error when trying to load the page.

 
 

Drupal is a registered trademark of Dries Buytaert.