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 
Command icon 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:

  • 2.x Comparechanges, plain diff MR !1

Comments

Sergiu Stici created an issue. See original summary.

sergiu stici’s picture

Status: Active » Needs review
StatusFileSize
new1.37 KB

Here is the patch, please review.

nickdickinsonwilde’s picture

StatusFileSize
new27.59 KB

Since 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.

nickdickinsonwilde’s picture

jussni’s picture

StatusFileSize
new634 bytes

Hi, 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 127

After changing the imported class, it seems to work.

bryandenijs’s picture

Thanks 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.

nickdickinsonwilde’s picture

Status: Needs review » Reviewed & tested by the community
robin.ingelbrecht’s picture

@NickDickinsonWilde I'm currently looking for a co-maintainer. Would you be interested in co-maintaining?

nickdickinsonwilde’s picture

@robin.ingelbrecht. I'd definitely be interested - be great to have a D9 version out!
Thanks
Nick

robin.ingelbrecht’s picture

@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? :)

nickdickinsonwilde’s picture

yep, access confirmed, thanks @robin.ingelbrecht

nickdickinsonwilde’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 2.x branch, 2.0 release coming soon

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.