any plan for Drupal 6?

bgg - May 17, 2008 - 03:49
Project:Signwriter
Version:HEAD
Component:Code
Category:feature request
Priority:critical
Assigned:Unassigned
Status:closed
Description

would love to see it run.. on D6

#1

catorghans - May 26, 2008 - 14:46
Status:active» needs review

I made a Drupal 6 version.

I am not using all it's functionality AND it's my first drupal 6 module, so please test carefully.

AttachmentSize
signwriterD6.zip 17.42 KB

#2

macrocosm - July 12, 2008 - 07:03

This appears to be only partially working ... at least for me.

First, I could not seem to resolve the font path on my test environment, which unfortunately is a WAMP setup.
Had to move it to my online LAMP server to get the paths to resolve correctly.

Now all I see is a black box in place of my replaced title ... and in the cache directory those created images are all black as well.

I'm guessing it may just be my font or some issue with the GD library.

Not sure if anyone else had similar issues...
I will keep pecking away at it and post back here what I find.

Thanks for starting this port!!!

#3

macrocosm - July 12, 2008 - 09:47

I think its my choice of background color... #E8DEC
For some reason it wont work with that and creates a black image instead.
Strange and annoying ... as thats the color of my main page elements which the titles lay atop.
Im so close! It works when I make the background white! lol

#4

macrocosm - July 12, 2008 - 09:51
Status:needs review» reviewed & tested by the community

OMG im so lame!

lol I didnt even notice thats only 5 characters instead of E8DEC1 which it should be.
.....thinking lack of sleep is catching up to me!
Dohh!

So yea I can confirm that your patch works and is ready for prime-time!

#5

catorghans - July 13, 2008 - 19:10

great.

I used it on my second site, where it also worked as it should: www.hbll.nl
(the top menu is signwritered)

#6

macrocosm - July 14, 2008 - 21:26

Nice site ... were you able to set some sort of anti-aliasing? Your fonts seem to come out very clean.
Thanks again for this most excellent D.6 Port!
^_^

#7

catorghans - July 16, 2008 - 07:23

I always use the background color of the site as the background color of the image (even if I make the background transparent)

#8

naquah - August 3, 2008 - 15:14

Before checking this in, note that catorghans has reordered some of the functions, and that the files have windows line endings.

#9

catorghans - August 10, 2008 - 11:59

thanks, the line endings will not happen again (switched to linux)

#10

macrocosm - August 16, 2008 - 22:39

Is there some particular way to handle block titles in D6?
I have been able to make filters as well as replace my page titles ... but for some reason im not able to get the block titles done with the example provided which I have below.

<?php
$signwriter
= signwriter_load_profile('block_title');
$signwriter->text = $block->subject;
$url = signwriter_url($signwriter);
print
'<img src="'. signwriter_url($signwriter) .'" alt="'. $block->subject .'" />';
?>

If I figure it out I will post back here.

#11

pillarsdotnet - September 3, 2008 - 05:06
Status:reviewed & tested by the community» needs review

Ahem! A zipfile containing the complete edited module is *not* a patch.

I have compared your edited version with today's cvs HEAD checkout and minimized the number of changes.

The resultant patch is attached; I have not tested it, so I am changing the status back to "code needs review".

AttachmentSize
HEAD-to-6.x-upgrade.diff 16.27 KB

#12

Justin W Freeman - September 8, 2008 - 23:10

There is now a 6 dev version up which has this patch with some bug fixes plus the most recent changes that were done to the 5 version.

It seems to be stable but if peolpe can test it out that would be great.

Thanks.

#13

Justin W Freeman - September 8, 2008 - 23:15
Status:needs review» fixed

#14

Anonymous (not verified) - September 22, 2008 - 23:22
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.