diff --git a/dev/TestRunner.php b/dev/TestRunner.php index 43fb0fc65..805c6523a 100644 --- a/dev/TestRunner.php +++ b/dev/TestRunner.php @@ -58,7 +58,6 @@ class TestRunner extends Controller { * Run all test classes */ function all() { - die("here"); if(hasPhpUnit()) { $tests = ClassInfo::subclassesFor('SapphireTest'); array_shift($tests); @@ -127,10 +126,8 @@ class TestRunner extends Controller { $results = new PHPUnit_Framework_TestResult(); $results->addListener($reporter); - /*, array("reportDirectory" => "/Users/sminnee/phpunit-report")*/ if($coverage) { $suite->run($results); - //$testResult = PHPUnit_TextUI_TestRunner::run($suite, array("reportDirectory" => "../assets/coverage-report")); $coverageURL = Director::absoluteURL('assets/coverage-report'); echo "

Coverage report available here

"; } else { diff --git a/forms/FieldGroup.php b/forms/FieldGroup.php index 6eb265948..96dd4ef68 100755 --- a/forms/FieldGroup.php +++ b/forms/FieldGroup.php @@ -57,18 +57,16 @@ class FieldGroup extends CompositeField { * it is easier to overwrite the
behaviour in a more specific class */ function Field() { - $subfieldParam = $this->subfieldParam; - $fs = $this->FieldSet(); $spaceZebra = isset($this->zebra) ? " $this->zebra" : ''; $idAtt = isset($this->id) ? " id=\"{$this->id}\"" : ''; - $content = ""; + $content = "
"; foreach($fs as $subfield) { - $childZebra = (!isset($childZebra) || $childZebra == "odd") ? "even" : "odd"; - if($subfield->hasMethod('setZebra')) $subfield->setZebra($childZebra); - $content .= $subfield->$subfieldParam() . " "; + $childZebra = (!isset($childZebra) || $childZebra == "odd") ? "even" : "odd"; + if($subfield->hasMethod('setZebra')) $subfield->setZebra($childZebra); + $content .= "
" . $subfield->{$this->subfieldParam}() . "
"; } - $content .= ""; + $content .= "
"; return $content; } diff --git a/security/Member.php b/security/Member.php index 9f31c2f9a..c50ac92de 100644 --- a/security/Member.php +++ b/security/Member.php @@ -699,7 +699,7 @@ $_REQUEST['showqueries'] = 1; public function Groups() { $groups = $this->getManyManyComponents("Groups"); - $unsecure = DataObject::get("Group_Unsecure", ""); + $unsecure = DataObject::get("Group_Unsecure"); if($unsecure) { foreach($unsecure as $unsecureItem) { $groups->push($unsecureItem);