php drupal-check.phar modules/contrib/single_page_site/
8/8 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
------ ----------------------------------------------------
Line src/Controller/SinglePageSiteController.php
------ ----------------------------------------------------
77 Call to deprecated function drupal_get_messages().
120 Call to deprecated function drupal_set_message().
------ ----------------------------------------------------
------ ---------------------------------------------------
Line src/Form/SinglePageSiteConfigForm.php
------ ---------------------------------------------------
232 Call to deprecated function drupal_set_message().
------ ---------------------------------------------------
[ERROR] Found 3 errors | Comment | File | Size | Author |
|---|---|---|---|
| #6 | imported_class_error.patch | 634 bytes | jussni |
| #3 | 3084146-drupal-9-support-3.patch | 27.59 KB | nickdickinsonwilde |
| #2 | deprecated-3084146-2.patch | 1.37 KB | sergiu stici |
Issue fork single_page_site-3084146
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
sergiu stici commentedHere is the patch, please review.
Comment #3
nickdickinsonwildeSince September, there have been.. a lot of changes.
New patch attached basically ignoring the old patch, so no interdiff. Due to symphony changes/core changes I recommend releasing this as a new major version - as with this patch it is *only* Drupal 9 compatible.
Comment #4
nickdickinsonwildeComment #6
jussni commentedHi, I have tested your d9.x-1.x branch and encountered this issue:
Got error 'PHP message: TypeError: Argument 2 passed to Drupal\\Component\\EventDispatcher\\ContainerAwareEventDispatcher::dispatch() must be an instance of Symfony\\Component\\EventDispatcher\\Event or null, instance of Drupal\\single_page_site\\Event\\EventSinglePageSiteAlterOutput given, called in /modules/single_page_site/src/Controller/SinglePageSiteController.php on line 127After changing the imported class, it seems to work.
Comment #7
bryandenijsThanks for this new branch.
And Thanks Jossni, your patch fixes my problem.
Now I can go and test the rest of my website after the D9 update.
Comment #8
nickdickinsonwildeComment #9
robin.ingelbrecht commented@NickDickinsonWilde I'm currently looking for a co-maintainer. Would you be interested in co-maintaining?
Comment #10
nickdickinsonwilde@robin.ingelbrecht. I'd definitely be interested - be great to have a D9 version out!
Thanks
Nick
Comment #12
robin.ingelbrecht commented@NickDickinsonWilde I added you as a maintainer. You should be able to maintain issues and releases as well as pushing to the GIT repo.
Can you confirm? :)
Comment #13
nickdickinsonwildeyep, access confirmed, thanks @robin.ingelbrecht
Comment #14
nickdickinsonwildeCommitted to 2.x branch, 2.0 release coming soon