Download & Extend
Technorati » Issues

Update Technorati Module to Drupal 6

Posted by nexy on June 30, 2008 at 6:26pm
Project:Technorati
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:jandd
Status:closed (fixed)

Issue Summary

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

Thanks

Login or register to post comments

Comments

#1

Posted by kbahey on June 30, 2008 at 6:38pm
Title:Update Technorati Module» Update Technorati Module to Drupal 6

This question is frequently asked.

#2

Posted by nexy on July 2, 2008 at 1:02am

thank you for the link

#3

Posted by jandd on July 16, 2008 at 6:09am
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

Posted by jandd on July 27, 2008 at 3:47pm
Version:5.x-1.x-dev» master
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

Posted by kbahey on July 27, 2008 at 4:34pm
Version:master» 6.x-1.x-dev
Status:needs review» fixed

Thanks!

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

#6

Posted by Anonymous (not verified) on August 10, 2008 at 4:43pm
Status:fixed» closed (fixed)

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

#7

Posted by kbahey on September 19, 2008 at 3:22pm

@jandd

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

#8

Posted by jandd on September 19, 2008 at 8:08pm

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).

nobody click here