simplyGits / Magister6.php

Easy to use, flexible PHP library for the Magister 6 API
GNU Lesser General Public License v3.0
4 stars 2 forks source link

Probleem met ophalen van roepnaam, achternaam en foto #5

Closed splintersuidman closed 8 years ago

splintersuidman commented 8 years ago

Hoi,

Ik ben bezig met het maken van een site die gegevens uit magister kan halen. Maar ik heb een probleempje. Als ik de volgende code heb, krijg ik wel de Magister-versie maar niet de roepnaam, achternaam en foto. (user en pass heb ik vervangen door mijn eigen gebruikersnaam en wachtwoord).

<html>
<body>
<?php
require('lib/Magister6.class.php');
$magister = new Magister("cgu.magister.net", "user", "pass");
// $magister->setSchool('https://cgu.magister.net/');
// $magister->setCredentials("user", "pass");
// $userInfo = $magister->getUserInfo();
// $magister->login();

echo('<h1>'Magister '.$magister->getMagisterInfo()->ProductVersie.'</h1>);

echo('Naam: '.$magister->getUserInfo()->Roepnaam.' '. $magister->getUserInfo()->Achternaam.'<hr>');

echo('<img src="data:image/jpeg;base64,'.$magister->getPicture(200,200,true).'">');

?>
</body>
</html>

Als ik

 // $magister->setSchool('https://cgu.magister.net/');
 // $magister->setCredentials("user", "pass");
 // $userInfo = $magister->getUserInfo();
 // $magister->login();

zonder // gebruik, krijg ik de error 'Session not verified'.

Is mijn code onjuist? Of hoe kan ik het anders oplossen?

Splinter

ghost commented 8 years ago

Bij de eerste log je niet in (probeer true als 4e argument toe te voegen bij de constructor)

Bij de tweede... ik heb dat probleem ook! Magister zegt dat de inlogcodes niet kloppen, maar als ik het zo'n 10x achter elkaar doe werkt 'ie. (Ik neem aan dat dat het probleem is, je kan altijd echoen wat je terug krijgt van Magister om zeker te zijn)

On 15:46, Wed, Jun 15, 2016 splintah notifications@github.com wrote:

Hoi,

Ik ben bezig met het maken van een site die gegevens uit magister kan halen. Maar ik heb een probleempje. Als ik de volgende code heb, krijg ik wel de Magister-versie maar niet de roepnaam, achternaam en foto. (user en pass heb ik vervangen door mijn eigen gebruikersnaam en wachtwoord).

`

<?php require('lib/Magister6.class.php'); $magister = new Magister("cgu.magister.net", "user", "pass"); // $magister->setSchool('https://cgu.magister.net/'); // $magister->setCredentials("user", "pass"); // $userInfo = $magister->getUserInfo(); // $magister->login();

echo(' Magister '.$magister->getMagisterInfo()->ProductVersie.'');

echo('Naam: '.$magister->getUserInfo()->Roepnaam.' '.

$magister->getUserInfo()->Achternaam.'

');

echo('

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/simplyGits/Magister6.php/issues/5, or mute the thread https://github.com/notifications/unsubscribe/AF-iZB3ds_FLslbpJZAgxU5bj5N4k0I-ks5qMAI_gaJpZM4I2X4l .

splintersuidman commented 8 years ago

Als ik als 4de parameter 'true' toevoeg, krijg ik ook de error 'Session not verified. Ik denk dat het probleem ligt bij het inloggen. Ook als ik $magister->login(); gebruik krijg ik de error.

splintersuidman commented 8 years ago

Hoe bedoel je "als ik het zo'n 10x achter elkaar doe"? En hoe echo ik alles wat ik van Magister terug krijg?

ghost commented 8 years ago

Ik heb de login in een while-loop gezet binnen een trycatch - zodat hij - als hij faalt - het maximaal 20x probeert. Meestal werkt hij na 10 keer.

On 16:30, Wed, Jun 15, 2016 splintah notifications@github.com wrote:

Hoe bedoel je "als ik het zo'n 10x achter elkaar doe"?

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/simplyGits/Magister6.php/issues/5#issuecomment-226204967, or mute the thread https://github.com/notifications/unsubscribe/AF-iZDk8U-MjicxVD8WHC2ov6BjsTJoRks5qMAxugaJpZM4I2X4l .

splintersuidman commented 8 years ago

@Codicassert zou je jouw code kunnen sturen?

tkon99 commented 8 years ago

Hi allemaal, Het probleem is bekend bij mij. Het is sinds de laatste magister api update, heb er nog niet naar gekeken. Zal het dit weekend inplannen.

splintersuidman commented 8 years ago

@tkon99 oké fijn!

tkon99 commented 8 years ago

@lieuwex hadden we dit bij magisterjs ook niet?

ghost commented 8 years ago

Oh ja, die laatste update zorgt ervoor dat je JSON MOET versturen (i.p.v.) formdata

On 16:47, Wed, Jun 15, 2016 Thomas Konings notifications@github.com wrote:

Hi allemaal, Het probleem is bekend bij mij. Het is sinds de laatste magister api update, heb er nog niet naar gekeken. Zal het dit weekend inplannen.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/simplyGits/Magister6.php/issues/5#issuecomment-226210638, or mute the thread https://github.com/notifications/unsubscribe/AF-iZEUnr3XF_geADI6AYXy6IH9pXB9Kks5qMBCVgaJpZM4I2X4l .

tkon99 commented 8 years ago

Het is trouwens niet verboden om een pull-request te maken :wink:

lieuwex commented 8 years ago

@tkon99 We hadden geen problemen met Magister 6.1, maar wel met een eerdere versie. Waarvoor ik de inlogcode toen heb aangepast.

tkon99 commented 8 years ago

@lieuwex Thanks!

ghost commented 8 years ago

Was op school, zal als ik thuis ben mijn inlogcode in een PR zetten :P (als je er niet dan al eentje hebt)

On 16:54, Wed, Jun 15, 2016 Thomas Konings notifications@github.com wrote:

Het is trouwens niet verboden om een pull-request te maken 😉

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/simplyGits/Magister6.php/issues/5#issuecomment-226212739, or mute the thread https://github.com/notifications/unsubscribe/AF-iZDKP21XAxYAybL3jJtwfWGZmQB7Nks5qMBIRgaJpZM4I2X4l .

ghost commented 8 years ago

https://github.com/simplyGits/Magister6.php/pull/6 PR gemaakt :ok_hand:

tkon99 commented 8 years ago

Zou nu gefixt moeten zijn. Mocht er iets voorvallen, open dan gerust een nieuwe issue.