Should we decouple any modules

Created on 25 April 2024, 7 months ago
Updated 15 August 2024, 3 months ago

Problem/Motivation

I'm evaluating using quiz for a project (last time I used it was Drupal6!)

I see 6.x branch has LOTS of dependencies: entity, range, rules, typed_data, paragraphs, entity_reference_revisions, views_bulk_operations, views_data_export, rest, csv_serialization

Are all of these really required? I'm pretty sure some could be optional. Specially concerned about rules (no stable version) and paragraphs (I don't use paragraphs on my site, not sure why this is required for quiz). Also VBO, data export and csv serialization looks like optional to me.

Steps to reproduce

Proposed resolution

Maybe they should be optional? Maybe they are test dependencies?

Remaining tasks

User interface changes

API changes

Data model changes

🌱 Plan
Status

Active

Version

7.0

Component

Code - Quiz core

Created by

πŸ‡ͺπŸ‡ΈSpain penyaskito Seville πŸ’ƒ, Spain πŸ‡ͺπŸ‡Έ, UTC+2 πŸ‡ͺπŸ‡Ί

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @penyaskito
  • πŸ‡ΊπŸ‡ΈUnited States djdevin Philadelphia

    Quiz builds on a lot of core/contrib functionality instead of reinventing (if you remember, was done for the D5-D7 versions, resulting in a mountain of technical debt).

    Rules is used for evaluating all the feedback items and allows for custom conditions. Agreed it should probably move to something like ECA or core conditions.

    entity - Quiz requires some Entity functionality not provided by core, e.g base entity classes/forms/handlers
    range - Needed for widgets like grade range (e.g. grade between X and Y)
    rules, typed_data - Needed for evaluating Feedback conditions
    paragraphs - Needed for all sorts of nested data storage e.g. MCQ questions, Feedbacks, Grade feedback, Taxonomy questions...
    entity_reference_revisions - Dependency of paragraphs, also powers Quiz revisioning
    views_bulk_operations - Needed for admin functionality e.g. question bank, quiz questions, results
    views_data_export, rest, csv_serialization - Needed for reporting/exports of Quiz results

  • πŸ‡ͺπŸ‡ΈSpain penyaskito Seville πŸ’ƒ, Spain πŸ‡ͺπŸ‡Έ, UTC+2 πŸ‡ͺπŸ‡Ί

    Thanks for your detailed answer. I need to mature this a bit, but wondering if we could decouple those.
    My usecase is: I want editors to create simple quizzes (probably only multianswer questions), and people (in my case anonymous, but don't think is relevant) would complete them and see how they did at the end.

    Multianswer answers being paragraphs is quite a lot, but I understand the reasoning. Could be simpler though.
    I won't have "evaluator" feedback (so paragraphs, rules and their dependencies are overkill for me).
    I need feedback to be custom based on results (so range should be in I think).
    I don't want to export the results (no views_data_export, rest, csv_serialization). Actually I was exploring how we could swap the quiz results storage, because I don't even want to store them if possible, even I understand we need those while the users are completing the quiz.

    It would be a pity ending with a custom solution or a mostly-duplicated "simplequiz" module because of those constraints.

  • πŸ‡ΊπŸ‡ΈUnited States smustgrave

    I'd be a +1 for this.

    Example not sure if field_group, vbo, or data_export are needed. People could always download them and update their views or displays themselves but takes the burden off quiz.

    Would need to happen in a new major release (7.0.x) and we could update the README with suggestion modules/.

  • πŸ‡ΊπŸ‡ΈUnited States smustgrave

    Personally I'd like to decouple VBO and data export from quiz. It's 100% useful but not every site uses the export and feel if someone

    I think field_group can be removed but made a dependency of quiz_multichoice. So if a site wants to install mutichoice they go out and download field_group

    Same deal for range and quiz_feedback

    Would have to research entity if that's needed

    Paragraphs seems to be needed.

    100% support replacing rules with ECA.

    Thoughts?

  • πŸ‡ΊπŸ‡ΈUnited States smustgrave

    So I am decoupling views data export. Module appears relatively unsupported and is a sole blocker for D11

Production build 0.71.5 2024