×

Notice

The forum is in read only mode.
Welcome, Guest
Username: Password: Remember me
  • Page:
  • 1

TOPIC: Captcha URL Bug

Captcha URL Bug 14 years 9 months ago #5412

  • Jeff
  • Jeff's Avatar Topic Author
  • Offline
  • Fresh Boarder
  • Fresh Boarder
  • Posts: 7
  • Thank you received: 0
I installed your component today and it is really nice, thank you!

I did find a little problem with the captcha image URL, please note the double forward slashes (//) after the domain name.

www.mysite.com //components/com_comment/joscomment/captcha.php?refid=a4e63deb3e895bb01cc0c1fdf8f97e06

I have Joomla's default SEF turned on (with mod rewrite) and this caused a problem on my site. Basically the captcha would not appear.

My URL rewrite does not rewrite anything in the /components folder. I had to modify my rewrite to include //components and then the captcha appeared!

Anyway, just thought I'd post this in case others have experienced similar issues due to the double forward slashes.

Thank you

Captcha URL Bug 14 years 9 months ago #5414

  • Daniel Dimitrov
  • Daniel Dimitrov's Avatar
  • Offline
  • Administrator
  • Administrator
  • Posts: 9618
  • Karma: 155
  • Thank you received: 1081
Oh, thank you for the bug report. I will fix this in beta2 - thanks!

dev.compojoom.com/issues/show/56

Captcha URL Bug 14 years 9 months ago #5460

  • JonusC
  • JonusC's Avatar
  • Offline
  • Platinum Boarder
  • Platinum Boarder
  • Posts: 785
  • Thank you received: 48
I took care of it. That took me five minutes :P

If you'd like to fix it yourself right now Jeff, open up this file...
yourwebsite.net/components/com_comment/joscomment/security.php
...and go to line number 30 or around there, just search for the text ?refid= as it might have changed between Beta1 and my local trunk copy.

Change the line from...
."<img src="". JURI::base() . "/components/com_comment/joscomment/captcha.php?refid=" . $refid . "" alt="Security Image" />n"
...to...
."<img src="". JURI::base() . "components/com_comment/joscomment/captcha.php?refid=" . $refid . "" alt="Security Image" />n"

Well you might have noticed the /components there yourself, but I make a habit of being verbose in many things I say :laugh:

Captcha URL Bug 14 years 9 months ago #5644

  • Jeff
  • Jeff's Avatar Topic Author
  • Offline
  • Fresh Boarder
  • Fresh Boarder
  • Posts: 7
  • Thank you received: 0
Thank you Jonus for that fix.

I personally handled the problem with my URL rewrite filter before you posted the fix. I simply compensated for the double forward slash.

This way when I upgrade this excellent component and if the problem is fixed or is not fixed, I will be perfectly OK.

If the bug is fixed in the new version, I'll simply remove that double slash condition from my filter.

Captcha URL Bug 14 years 9 months ago #5727

  • JonusC
  • JonusC's Avatar
  • Offline
  • Platinum Boarder
  • Platinum Boarder
  • Posts: 785
  • Thank you received: 48
Indeed, it is already committed and the Beta 2 release will have the fix, you can see it here dev.compojoom.com/repositories/diff/joco...security.php?rev=221

:)
  • Page:
  • 1
Time to create page: 0.113 seconds