Drupal 5 Branch of Splash

Rob Loach - July 7, 2008 - 16:49
Project:Splash
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

The Drupal 5 branch of Splash is kind of messed. 5.x-1.x-dev is tailored to HEAD, which looks like its made for Drupal 6. That release node should point to the DRUPAL-5 branch, which looks like it doesn't even exist. I'll put together a patch that will backport the latest Drupal 6 branch (DRUPAL-6--2) to a Drupal 5 version.

#1

Rob Loach - July 7, 2008 - 21:41
Status:active» needs review

I'd be willing to help maintain/create the Drupal 5 branch of Splash. The attached is a patch from DRUPAL-6--2-3 to backport it to Drupal 5 for a newly created DRUPAL-5.

AttachmentSize
splash_drupal_5.patch 20.08 KB

#2

dorien - September 22, 2008 - 10:42

Fantastic, just what I was looking for!

#3

dorien - September 26, 2008 - 13:30

Huh... If I patch spash 6.2.3 it gives me:
"Reversed (or previously applied) patch detected!"
But I continued...

The module still said 6.2.3 in the build/modules page and after enabling it, I got the following message:

    * warning: Missing argument 2 for splash_help() in /home/rawvegan/public_html/sites/all/modules/splash/splash.module on line 200.
    * warning: Missing argument 2 for splash_help() in /home/rawvegan/public_html/sites/all/modules/splash/splash.module on line 200.

#4

Rob Loach - September 26, 2008 - 20:28
Status:needs review» fixed

This is in already! http://drupal.org/node/192759

Additional patches welcome.

#5

Anonymous (not verified) - October 10, 2008 - 20:32
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.