Initial patch to port Textimage to Drupal 6.

Haven't tested Captch integration, but standard Textimage functionally tested and appears to work perfectly.

CommentFileSizeAuthor
#6 textimage-6.0-262708-3.patch36.19 KBdeciphered
#3 textimage-6.0-262708-2.patch58.2 KBdeciphered
textimage-6.0-1.patch57.86 KBdeciphered

Comments

wundo’s picture

wundo commented

Captcha integration is deprecated, thanks, I gotta review it.

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented

Thanks for the quick response.

From the extensive testing I've done so far I can confirm that there is atleast one isse, on line #692. I will be looking into it today if I have time.

The things I have tried and confirmed work:
- Create a preset
- Confirmed preset works
- Create a second preset that uses the first preset as a background
- Confirmed preset works
- Delete preset
- Flush presets
- Unnecessary as files are not being saved as per error on line #692

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented
StatusFileSize
new textimage-6.0-262708-2.patch58.2 KB

Updated Patch:
- Fixed permission for viewing of Textimages
- Fixed bug at #692 so files will be save properly so they are not recreated each time

encho’s picture

encho commented

subscribe

encho’s picture

encho commented

Which version it is patched against? Not working here.

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented
Version: 5.x-2.x-dev » 5.x-2.0
StatusFileSize
new textimage-6.0-262708-3.patch36.19 KB

Updated Patch:
- Patch is now rolled against 5.x-2.0 as it was more recent than HEAD
- Watchdog fixes
- Upgrade path is back

appel’s picture

appel commented

Subscribing.

sinuz’s picture

sinuz commented

subscribing

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented

@wundo

Curious if you've had time to look at this?
It's been a few months now and I have been using it all that time with no ill effects.

Paul Natsuo Kishimoto’s picture

Paul Natsuo Kishimoto commented

Subscribing!

chipway’s picture

chipway commented

Hi,

Why not a dev version for testing by many people if it work since a long time with no problem ?

Thank you

attiks’s picture

attiks commented

I'm using this as well on a couple of sites without any problems, can some release this?

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented

Just a quick update, 6.x-1.x-dev will be released shortly after #333711: Rename textimage HEAD release to 5.x-2.x-dev is done.

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented
Status: Needs review » Closed (fixed)

Committed