Tutorialurl issue



Acceptance Criteria


Thomas Marstrander
December 28, 2016, 1:40 PM

Yeah, that makes sense. good

Pål Jørgensen
December 28, 2016, 10:22 AM

This warning may be seen on all plugins out there. Therefore I added a fix that will fix it for all versions of the plugin...

I was first looking into returnning null, but I don't think that will work.

Thomas Marstrander
December 28, 2016, 9:10 AM

Looks like there already was a fix for it. Is it necessary to update the json with empty tutorialUrl objects ? This will cause the empty tutorialUrls to be stored in the db.

Anyway it is working well, good job!

Pål Jørgensen
December 27, 2016, 10:59 AM

: You fixed the client part here: https://github.com/h5p/h5p-php-library/commit/a326066c97dd4d226f2bc72cdfc5410438704918

I will look at the h5p.org-backend



Pål Jørgensen


Thomas Marstrander

Code reviewer

Thomas Marstrander