Skip to main content

This site requires you to update your browser. Your browsing experience maybe affected by not having the most up to date version.

Archive

Our old forums are still available as a read-only archive.

Moderators: martimiz, Sean, biapar, Willr, Ingo, simon_w

Patch: Fixes undefined variables in sapphire/forms


Reply

5 Posts   1928 Views

Avatar
elijahlofgren

16 May 2007 at 9:44am Google Summer of Code Hacker, 222 Posts

This patch fixes some undefined variable errors that I found and fixed in sapphire/forms: http://www.elijahlofgren.com/silverstripe/patches/Fix-undefined-form-vars-silverstripe.2.0.DailyBuild.2007-05-15.patch

Hope this helps,

Elijah

Avatar
Andy

16 May 2007 at 12:08pm 230 Posts

Excellent, applied :)

Avatar
elijahlofgren

17 May 2007 at 8:44am Google Summer of Code Hacker, 222 Posts

It looks like 1 line was missed when this patch was applied.

I get this error:
"Notice: Undefined variable: Type in /var/www/silverstripe/sapphire/forms/CheckboxField.php on line 38"

This 1 line change fixes the that error:
http://www.elijahlofgren.com/silverstripe/patches/Fix-Checkbox-Type-Error-silverstripe.2.0.DailyBuild.2007-05-16.patch

Avatar
elijahlofgren

24 May 2007 at 10:28am Google Summer of Code Hacker, 222 Posts

:: bump :: :)

In the latest daily build: silverstripe.2.0.DailyBuild.2007-05-23

I still need to apply this patch to fix 4 errors like:
"Notice: Undefined variable: Type in /var/www/silverstripe/sapphire/forms/CheckboxField.php on line 38"

I'd appreciate it if you would apply this: http://www.elijahlofgren.com/silverstripe/patches/Fix-Checkbox-Type-Error-silverstripe.2.0.DailyBuild.2007-05-16.patch

Thanks,

Elijah

Avatar
Andy

28 May 2007 at 1:52pm 230 Posts

Sorry, I'll try keep more on top on this! Applied! :)