17488 Posts in 4473 Topics by 1978 members
|
Page:
1
|
Go to End | |
| Author | Topic: | 1489 Views |
-
Trivial patch: remove some warnings

24 April 2007 at 10:09am
Hey guys,
I have php showing all warning on my box, and there are a few that seem to pop up on all of the silverstripe pages. (this is on 2.0.1)
All of them are from using variables without making sure they exist.
in sapphire/core/manifestBuilder.php (line 30)
< || $_GET['buildmanifest'] || $_GET['flush'];
---
> || (isset($_GET['buildmanifest']) && $_GET['buildmanifest'])
> || (isset($_GET['flush']) && $_GET['flush']);in sapphire/core/control/ContentController.php (line 182)
> $archiveLink = "";in sapphire/core/control/ContentNegotiator.php (line 19)
< } else if($_GET['forceFormat']) {
---
> } else if(isset($_GET['forceFormat']) && $_GET['forceFormat']) {same file (line 27)
< if(!$q[$preference]) $q[$preference] = $format;
---
> if(!isset($q)) $q = array();
> if(!isset($q[$preference]) || !$q[$preference]) $q[$preference] = $format;in sapphire/main.php (line 19)
< if($_REQUEST['trace']) apd_set_pprof_trace();
---
> if(isset($_REQUEST['trace']) && $_REQUEST['trace']) apd_set_pprof_trace();Hope that helps,
--Will -
Re: Trivial patch: remove some warnings

9 May 2007 at 2:59pm
Cheers! I went through SilverStripe a few months ago and fixed as many php warnings as I could, but there are still a number that went unnoticed. If you notice anymore, please let me know.
| 1489 Views | ||
|
Page:
1
|
Go to Top |



