Set requirement on file for main module

Created on 27 June 2024, 6 months ago
Updated 2 August 2024, 5 months ago

Problem/Motivation

When installing the module, then go to the field form. You'll get a fatal error when the file module is not installed.

Steps to reproduce

Install Drupal with minimal profile.
Install Cloudflare Stream
Have a content type and try adding a field
You should see the error

Proposed resolution

Set requirement in the module for file.

Remaining tasks

Create MR

User interface changes

-

API changes

-

Data model changes

-

🐛 Bug report
Status

Closed: outdated

Version

3.0

Component

Code

Created by

🇧🇪Belgium tim-diels Belgium 🇧🇪

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

Merge Requests

Comments & Activities

  • Issue created by @tim-diels
  • Assigned to dtfabio
  • 🇧🇪Belgium dtfabio Ninove

    Hi Tim,

    I will check this out.

    Greetings,

    Fabio

  • First commit to issue fork.
  • Merge request !37Set requirement in the module for file → (Open) created by abhiyanshu
  • Pipeline finished with Success
    6 months ago
    Total: 148s
    #209606
  • 🇮🇳India abhiyanshu

    @tim-diels,
    I have set the requirement in the module for the file. Please refer to the screenshot for more clarity. Thanks.

  • Status changed to Needs review 6 months ago
  • 🇧🇪Belgium dtfabio Ninove

    Hi abhiyanshu,

    Thanks for doing the work that needed to be done, only I did already mention I was going to take this up.

    In the future, it is best that you first check if someone else has not already specified that they are working on this before starting the ticket.

    This also ensures no duplication of work, which is certainly important for larger tickets.

    It is also best practice to transfer the ticket to "Needs review" after the work has been done, so other developers know it needs to be looked at.

    I will test your solution.

    Greetings,

    Fabio

  • Issue was unassigned.
  • Status changed to RTBC 6 months ago
  • 🇧🇪Belgium dtfabio Ninove

    Hi Tim,

    I tried the code change and the site now asks to turn on file and I no longer get a fatal error.

    Greetings,

    Fabio

  • 🇮🇳India abhiyanshu

    @dtfabio, I waited for a while since the changes were not that big, so I went ahead and pushed them.
    I will make sure to follow up on those things next time. Thank you for your valuable suggestions and the review. I appreciate your guidance!

  • 🇧🇪Belgium tim-diels Belgium 🇧🇪

    Hi abhiyanshu,

    To be frankly, you waited for 26 minutes. So you don't give a chance for someone to set up the project and do the code changes ...
    Thats not very friendly from your side. But as we're a friendly community and I was working together with Fabio to pick this up, I gave the advice to make you aware of it. I don't think your response is that professional so maybe something to work on next time you work on issues.

    Thank you both for the work and tests. I'll get this comitted.

  • Status changed to Closed: outdated 5 months ago
  • 🇧🇪Belgium tim-diels Belgium 🇧🇪

    Seems we fixed the dependency in another issue 📌 Transform the architecture to use a dedicated streamwrapper Fixed so don't need this anymore.

Production build 0.71.5 2024