Ik denk bij de open() method. Laat je error even zien hier anders via copy/paste.
Het is niet echt wenselijk en screenshots van foutmeldingen te plaatsen. Het is zeer onhandig.
<?php
// De constructor
public function __construct($id)
{
$this->id = $id;
return $this;
}
?>
De regel met de return $this; mag je bij een constructor achterwege laten. De constructor kan namelijk geen waarde teruggeven. Zou het een normale functie zijn dan doe je de return $this; enkel als je "Method chaining" wilt gebruiken én je functie verder niets teruggeeft (zoals bij Setters het geval is). Bij method chaining kun je verschillende functies achter elkaar koppelen. zoals het voorbeeld hieronder:
<?php
$user
->setName('Jan Smit')
->setBirthdate('31-12-1985')
->setCity('Volendam')
;
?>
Doordat setName(), setBirthdate() en setCity() de instantie van de class teruggeven kan je meteen de volgende setter aanroepen. Zou dat niet zo zijn dan zou je er drie verschillende statements van moeten maken:
<?php
$user->setName('Jan Smit');
$user->setBirthdate('31-12-1985');
$user->setCity('Volendam');
;
?>
Ik ben er ook niet helemaal zeker van of dit goed gaat:
<?php
return "circles/$this->id/";
?>
Ik zou de variabelen buiten de string laten:
<?php
return 'circles/' . $this->id . '/';
?>