Patch for RT-Extension-ResetPassword

Please find attached a patch for RT-Extension-ResetPassword.

This patch corrects two spots that do not build links correctly.

The links now obey $RT::WebURL

Jason ‘axelilly’ Fenner
GTalk: axelilly@gmail.com

RT-Extension-ResetPassword.urlfix.patch (1.49 KB)

Please find attached a patch for RT-Extension-ResetPassword.
This patch corrects two spots that do not build links correctly.
The links now obey $RT::WebURL

Two issues, you’re interpolating RT::WebURL inside a single
quoted string which won’t work, and you really want $RT::WebPath
because these are RT internal links. Take a look at the patch
I applied to:
html/Callbacks/RT-Extension-ResetPassword/Elements/Login

-kevin

Please find attached a patch for RT-Extension-ResetPassword.
This patch corrects two spots that do not build links correctly.
The links now obey $RT::WebURL

Two issues, you’re interpolating RT::WebURL inside a single
quoted string which won’t work, and you really want $RT::WebPath
because these are RT internal links. Take a look at the patch
I applied to:
html/Callbacks/RT-Extension-ResetPassword/Elements/Login

-kevin


List info:
http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel

Thanks for pointing that out.

I have created a new patch with using $RT::WebPath and not interpolated
within a quoted string.

Jason ‘axelilly’ Fenner
GTalk: axelilly@gmail.com

RT-Extension-ResetPassword.urlfix.2.diff (1.5 KB)

I have created a new patch with using $RT::WebPath and not
interpolated within a quoted string.

I’ve applied this and I’ll ship a version 0.03 to CPAN

-kevin