3 Notices removal

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@41720 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Romain Louis 2007-09-14 02:04:30 +00:00
parent 5a74d0e63d
commit 1ad4769c5f

View File

@ -212,7 +212,7 @@ JS;
// Get all the requests // Get all the requests
$ID = isset($_REQUEST['ctf']['ID']) ? Convert::raw2xml($_REQUEST['ctf']['ID']) : null; $ID = isset($_REQUEST['ctf']['ID']) ? Convert::raw2xml($_REQUEST['ctf']['ID']) : null;
$childID = Convert::raw2xml($_REQUEST['ctf']['childID']); $childID = isset($_REQUEST['ctf']['childID']) ? Convert::raw2xml($_REQUEST['ctf']['childID']) : null;
$childClass = Convert::raw2xml($_REQUEST['fieldName']); $childClass = Convert::raw2xml($_REQUEST['fieldName']);
$this->methodName = isset($_REQUEST['methodName']) ? $_REQUEST['methodName'] : null; $this->methodName = isset($_REQUEST['methodName']) ? $_REQUEST['methodName'] : null;
@ -225,12 +225,12 @@ JS;
// from the object via a string method call. // from the object via a string method call.
if(is_a($this->detailFormFields,"Fieldset")){ if(is_a($this->detailFormFields,"Fieldset")){
$detailFields = $this->detailFormFields; $detailFields = $this->detailFormFields;
} else if( $childData && is_string($this->detailFormFields)){ } else if( isset( $childData ) && is_string($this->detailFormFields)){
$functioncall = $this->detailFormFields; $functioncall = $this->detailFormFields;
if($childData->hasMethod($functioncall)){ if($childData->hasMethod($functioncall)){
$detailFields = $childData->$functioncall(); $detailFields = $childData->$functioncall();
} }
} elseif(!$childData || $this->methodName == 'add') { } elseif(! isset( $childData ) || $this->methodName == 'add') {
$SNG_sourceClass = singleton($this->sourceClass); $SNG_sourceClass = singleton($this->sourceClass);
if(is_numeric($ID) && $this->getParentClass()) { if(is_numeric($ID) && $this->getParentClass()) {
// make sure the relation-link is existing, even if we just add the sourceClass // make sure the relation-link is existing, even if we just add the sourceClass