Aufruf in Include-File funktioniert, außerhalb nicht?

[Rolf B]

Hallo Klaus1,

hast Du überprüft, was in beiden Fällen in $token, $url und $params steht? Aus meiner Sicht muss da etwas abweichen.

Es wäre auch sinnvoll, $token und $url nicht global abzulegen, sondern der Funktion als Argument zu übergeben. Global ist bäh.

Alternativ mach eine Klasse Moodle, wo token und url als Eigenschaften hinterlegt sind, und get_courses kann dann eine Methode davon sein. Mit Klassen tut man sich oft leichter, seine Daten sinnvoll zu strukturieren. Grundlegende Lektüre.

Ich weiß nicht, wo das Token herkommt. Klingt nach einem API-Schlüssel oder einer Art Session-ID - das Beschaffen dieses Tokens kann auch in einer Methode landen.

class Moodle {
   private $url;
   private $token;

   public function __construct($url = "https://moodel.dee.doo") {
      $this->url = $url;
   }

   public function getToken($blub) {
      $this->token = /* do some magic */;
   }

   public function getCourses() {
      return $this->getJsonData('core_course_get_courses')
   }

   private function getJsonData($methodName) {
      $params = [
         'wstoken' => $this->token,
         'wsfunction' => $methodName, 
         'moodlewsrestformat' => 'json'
      ];
      $url = $this->url . '?' . http_build_query($params);

      $ch = curl_init($url);
      if ($ch === FALSE) {
         // Hier Fehler loggen oder anderweit bekanntgeben.
         return null;
      }

      curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
      $response = curl_exec($ch);
      curl_close($ch);

      return json_decode($response, true);
   }
}

Ich habe das jetzt ungetestet runtergetippt, keine Garantie gegen Blödheitsfehler.

curl_setopt für die URL braucht man jedenfalls nicht, die kann man direkt an init übergeben.

Ich habe die Parameter-Erstellung, den Curl-Zugriff und die JSON-Dekodierung in eine Methode gesteckt. Das ist nicht unbedingt sinnvoll so. Es hängt davon ab, wie Du die Dinge kombinieren musst. Wenn jeder API Aufruf einen Parameter aus Token, Function und Format erwartet und immer JSON zurückgibt, kann man das so kombinieren. Andernfalls müsste man die Methoden stärker aufgliedern - vermutlich braucht es dann eine flexiblere Erstellung des Parameterarrays. Da kann ich Dir nur Rat geben, wenn ich mehr Methoden von Dir kenne. Wenn Du auch Dinge an Moodle POSTen willst, sieht ja auch der CURL anders aus, dann brauchst Du für Curl-Post eine eigene Methode.

Jedenfalls sollten am Ende alle Moodle-Zugriffe in dieser Klasse gebündelt sein, so dass $token und $url sonst nirgends mehr gebraucht werden.

Die Verwendung (Annahme: Klasse steht in moodle.class.php) sähe so aus:

require_once("moodle.class.php");

$moodle = new Moodle("https://mein.moodle.de");
$moodle->getToken("blub");   // Was auch immer Du brauchst, um ein Token zu bekommen

$courses = $moodle->getCourses();

Rolf