Update Technorati Module to Drupal 6

nexy - June 30, 2008 - 18:26
Project:Technorati
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:jandd
Status:closed
Description

Hello, just wondering if there could be an update to this module for Drupal 6.x?

Thanks

#1

kbahey - June 30, 2008 - 18:38
Title:Update Technorati Module» Update Technorati Module to Drupal 6

This question is frequently asked.

#2

nexy - July 2, 2008 - 01:02

thank you for the link

#3

jandd - July 16, 2008 - 06:09
Assigned to:Anonymous» jandd

I'll port the module because I need it for an own Drupal installation I'd like to upgrade. Please wait for a patch.

#4

jandd - July 27, 2008 - 15:47
Version:5.x-1.x-dev» HEAD
Status:active» needs review

Hello,

the attached patch against CVS HEAD makes the Technorati module compatible with Drupal 6 (tested with 6.3).

In addition to the patch, the folder "po" needs to be removed and a new folder "translations" needs to be created.

The code in the patch has been checked with the Coder Code review functionality using the following settings:

  • Converting 5.x modules to 6.x
  • Drupal Coding Standards (every developer should use)
  • Drupal Commenting Standards (every developer should use)
  • Drupal SQL Standards (new review, so use with caution)
  • Drupal Security Checks (very basic, needs work, but what it finds is good)
  • Interface text translatability

apply the checked coding reviews was set to "minor (most)"

Coder detected no problems and the module works and can be installed and uninstalled cleanly. The uninstall functionality has been added because it was not available.

AttachmentSize
technorati-6.x-1.x-dev.patch 11.41 KB

#5

kbahey - July 27, 2008 - 16:34
Version:HEAD» 6.x-1.x-dev
Status:needs review» fixed

Thanks!

Release should appear here in 24 hours http://drupal.org/node/287809

#6

Anonymous (not verified) - August 10, 2008 - 16:43
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

#7

kbahey - September 19, 2008 - 15:22

@jandd

Any reason to used a minimum version of PHP of 5.1? Which code depends on this?

#8

jandd - September 19, 2008 - 20:08

its just the minimum version I had available for testing. The version can be lowered if someone could test the functionality (but 5.2 is recommended by the PHP developers and 4.x is at end of life).

 
 

Drupal is a registered trademark of Dries Buytaert.