Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP 7.4 error curly braces deprecated in require_once() (and Update to markdown 1.9) #4

Closed
jenlampton opened this issue Sep 3, 2021 · 8 comments

Comments

@jenlampton
Copy link
Member

jenlampton commented Sep 3, 2021

In PHP 7.4, I see the following error on my status report:

Deprecated function: Array and string offset access syntax with curly braces is deprecated in require_once() (line 10 of backdrop/modules/contrib/markdown/includes/Markdown.inc.php).

It looks like we may need to update to the 1.9 release of markdown.

See related:

@jenlampton jenlampton changed the title PHP 7.4 error curly braces deprecated in require_once() PHP 7.4 error curly braces deprecated in require_once() (Update to markdown 1.9) Sep 27, 2021
@jenlampton jenlampton changed the title PHP 7.4 error curly braces deprecated in require_once() (Update to markdown 1.9) PHP 7.4 error curly braces deprecated in require_once() Sep 30, 2021
jenlampton pushed a commit to jenlampton/markdown that referenced this issue Sep 30, 2021
@jenlampton
Copy link
Member Author

I discovered a syntax error in module_load_include() that seems to fix this issue. The update to 1.9.0 wasn't necessary, so I separated that out into a different issue.

@jenlampton
Copy link
Member Author

Well, I was wrong, we do need to update the library to avoid this error. I've included that commit on this PR and closed that other issue.

@jenlampton jenlampton changed the title PHP 7.4 error curly braces deprecated in require_once() PHP 7.4 error curly braces deprecated in require_once() (and Update to markdown 1.9) Sep 30, 2021
@bugfolder
Copy link
Collaborator

Maybe time for a merge of the PR and new release?

@alanmels
Copy link
Member

The module has not been touched for 6 years already. Are two of the maintainers still interested in the module?

  • Chris Herberte (https://github.com/chrisherberte)
    
  • Fredrik Jonsson (https://github.com/frjo)
    

@bugfolder
Copy link
Collaborator

@alanmels, do you want to adopt the module if it's been abandoned? (I will if you don't want to.)

There's an official process for determining if a module has been abandoned. I'll start that process by filing the issue.

@alanmels
Copy link
Member

@bugfolder, unfortunately no time, so please go ahead take over the maintainership of the module as the current ones do not seem to be reacting at all.

@alanmels
Copy link
Member

I've updated the library to 2.0.0 See new PR https://github.com/backdrop-contrib/markdown/pull/10/files

@bugfolder
Copy link
Collaborator

Merged the update to 2.0.0. Thanks, @alanmels!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants