Cross referencing #2300871: Quiz module integration

H5P could expand Quiz by providing more question types without us having to have the logic in Quiz.

This is only the issue to build in support for it.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

djdevin’s picture

djdevin’s picture

FileSize
5.21 KB

  • djdevin committed 790c3d9 on 7.x-5.x
    Issue #2383029 by djdevin: support for integrations such as h5p where...
djdevin’s picture

Title: H5P integration » Support for H5P integration

Committed just the code needed for H5P integration.

See #2397247: H5P integration for the actual integration.

  • djdevin committed a40ccf5 on 7.x-5.x
    Revert "Issue #2383029 by djdevin: support for integrations such as h5p...
djdevin’s picture

Status: Active » Needs review

Forgot to test this.

Status: Needs review » Needs work

The last submitted patch, 2: h5p_integration-2383029-2.patch, failed testing.

djdevin’s picture

Status: Needs work » Needs review
FileSize
2.24 KB
djdevin’s picture

Issue summary: View changes
Status: Needs review » Fixed

  • djdevin committed 51ea845 on 7.x-5.x
    Issue #2383029 by djdevin: Support for H5P integration
    

Status: Fixed » Needs work

The last submitted patch, 8: support_for_h5p-2383029-8.patch, failed testing.

djdevin’s picture

Status: Needs work » Closed (fixed)
djdevin’s picture

falcon’s picture

I'm not sure if we should move it to a separate project until we have discussed this and thought this through. What will Quiz 6 be? Is there a need for separate question modules in Quiz if we can fill in the missing pieces in the H5P integration?

djdevin’s picture

The reason I put it in it's own project is because it's a "big" question type that deserves it's own project and maintainers. Just like DDLines, it's also a very specific question type that shouldn't hold Quiz core back. If we put every question type in Quiz core we'd end up having more and more to maintain and that stifles new releases. Quiz 4 was so far behind, so moving forward this would mean that we could go ahead and make Quiz 6 without H5P, DDlines, Cloze, etc. holding us back, and only keep the basic question types.

That seems to be the direction everything else is going as well, look at Ubercart/Commerce. Ubercart included about 10 payment gateways by default, but that also meant the Ubercart core project had to support them. Commerce doesn't include any payment gateways now.

trofiarik’s picture

Hi Devin,

is the integration between Quiz 5 and H5P still going on or the project has been abandoned?

I did not try Q4 but if it is true that it supports h5p integration, why is so difficult to implement it in the Q5?

Thanks,

Riccardo

nithinkolekar’s picture

@djdevin #13
module link posted doesn't have any module files nor at code repository. Is that module abandoned or committed patch #11 is enough?

djdevin’s picture

webservant316’s picture

I agree that a separate module is they way to go so that quiz can move ahead a bit more freely. However, one caveat is that for people like me who need H5P, then I am stuck at quiz 4.x until the quiz_h5p module begins to perform better. The separate quiz_h5p module allows quiz to move ahead more freely, but it also allows quiz_h5p to fall behind.