Error Flipbook settings

Created on 13 June 2024, 6 months ago

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

💬 Support request
Status

Active

Version

10.0

Component

User interface

Created by

🇪🇸Spain jrperezborges

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

Merge Requests

Comments & Activities

  • Issue created by @jrperezborges
  • 🇮🇳India kushal bansal

    Hi @jrperezborges

    I tried to reproduce this issue on 10.0.1, but eveything seems to be working fine. Can u provide steps to reproduce this issue.

    Thanks

  • Assigned to kushal bansal
  • First commit to issue fork.
  • Merge request !9Issue #3454487 → (Open) created by imclean
  • Status changed to Needs review 6 months ago
  • 🇦🇺Australia imclean Tasmania

    Error in db log when going to Admin -> Config -> User interface -> Flipbook Settings

    TypeError: Drupal\flipbook\Form\ChoosePdfStyleForm::__construct(): Argument #1 ($config) must be of type Drupal\Core\Config\ConfigFactoryInterface, Drupal\Core\Messenger\Messenger given, called in /web/modules/contrib/flipbook/src/Form/ChoosePdfStyleForm.php on line 52 in Drupal\flipbook\Form\ChoosePdfStyleForm->__construct() (line 39 of /web/modules/contrib/flipbook/src/Form/ChoosePdfStyleForm.php).

    The parameters supplied from create() are in a different order to what __construct() expects. I've also re-ordered the assigmnents in __construct() to match the order of the parameters.

  • 🇳🇱Netherlands Johan den Hollander

    Confirming the patch resolves the issue where the settingspage cannot be used.

  • 🇵🇭Philippines clarkssquared

    Hi

    I also applied MR !9 and I confirmed that the WSOD error is now fixed, please look at the monosnap screenshot in Before and After MR for your reference.

Production build 0.71.5 2024