Blogroll compatability with 4.7

eaton - November 25, 2005 - 21:05
Project:Blogroll
Version:HEAD
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

This patch updates blogroll.module for compatability with Drupal 4.7 and the new Form API.

It also makes a number of minor tweaks:

  1. The title of the blogroll block is now customizable
  2. The description displayed on the blogroll page is now customizable
  3. Link 'popularity' display can be turned on and off
  4. The 'edit' link now appears on the block for valid blogroll users no matter what page they're on.

The functional changes may not be desired -- if there's interest in the 4.7 patch but NOT in these feature changes, I can try to roll a smaller patch with only the update.

AttachmentSize
blogroll.module.patch13.76 KB

#1

eaton - November 25, 2005 - 21:22

Whoops. Copied an existing error in the module -- it checked for the user permission 'edit own blog' rather than 'create blog posts' or 'edit own blog posts'... Making impossible for users other than the Administrator to alter the blogroll.

Fixed.

AttachmentSize
blogapi.module_3.patch 788 bytes

#2

eaton - November 25, 2005 - 21:37

After some additional testing, it appears there are problems displaying the blogroll correctly in any situation where pathauto has been used to change the current URL. The problem existed previous to this patch, and so probably needs to be fixed separately.

Also, I attached the patch for a completely different module in my previous post. Durrrr.

AttachmentSize
blogroll.module_0.patch 13.84 KB

#3

mrmachine - April 25, 2006 - 02:29

I applied this patch to use with Drupal CVS, and everything looks alright except that when you add a link to your blogroll, nothing shows up ... it is not actually added to the blogroll.

#4

silurius - May 25, 2006 - 18:10

Add me to the list of those who would like to use this under 4.7.

#5

valley - January 2, 2007 - 07:13

Hello,
I have managed to install Blogroll on a Drupal 4.7 site
http://paratlane.net

The Block is active (left bottom) with a link "more"
I have my own blog set up but can't see this in the Block.

When you click on more a New page with Blogroll as title
appears but empty.

Tried to apply your latest patch (Nov 30) but get the
following error

++++++++++++++++++++++++++++
Hmm, not sure if this worked either. Here is the output:

[root@server modules]# patch -p0 -u < patch1.txt
can't find file to patch at input line 8
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|Index: modules/blogroll/blogroll.module
|===================================================================
|RCS file:
/cvs/drupal-contrib/contributions/modules/blogroll/blogroll.module,v
|retrieving revision 1.10
|diff -u -r1.10 blogroll.module
|--- modules/blogroll/blogroll.module 31 Jan 2005 02:37:41 -0000
1.10
|+++ modules/blogroll/blogroll.module 25 Nov 2005 20:54:52 -0000
--------------------------
File to patch: blogroll.module
patching file blogroll.module
Hunk #1 FAILED at 38.
Hunk #2 FAILED at 61.
Hunk #3 FAILED at 273.
Hunk #4 FAILED at 309.
Hunk #5 FAILED at 348.
5 out of 5 hunks FAILED -- saving rejects to file blogroll.module.

++++++++++++++++++++++++++++++++

You have included some unix comands for entering in the block
configuration.
Is there any simpler way as your instructions are a bit hard to
grasp . Please PM me if you need more info. I woulf very much like
to get this working on our site.

Many thanks

#6

valley - January 2, 2007 - 08:09

Hello I also get the following error when you open administer/settings/Blogroll
PHP Fatal error: Call to undefined function: form_select() in /home/paratlan/paratlan-www/modules/blogroll.module on line 64

Where Is this function defined ?

#7

robbiethegeek - September 23, 2009 - 14:16
Status:needs review» closed

Just took over this module and starting with Drupal 6 support only

 
 

Drupal is a registered trademark of Dries Buytaert.