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

ui-accordion open=0 gets an argument error #29

Closed
blueforce opened this issue Jan 7, 2019 · 14 comments
Closed

ui-accordion open=0 gets an argument error #29

blueforce opened this issue Jan 7, 2019 · 14 comments
Labels

Comments

@blueforce
Copy link

blueforce commented Jan 7, 2019

Steps to reproduce:

  1. User Shortcode Accordeon
  2. Set [ui-accordion independent=false open=0]
  3. see result

Failure:
Server Error
Sorry, something went terribly wrong!

0 - Argument 1 passed to Grav\Plugin\ShortcodeManager::getId() must implement interface Thunder\Shortcode\Shortcode\ShortcodeInterface, null given, called in root/user/plugins/shortcode-ui/shortcodes/AccordionsShortcode.php on line 33

Tested with:

  • UI Shortcode v2.2.5
  • PHP 7.2.x
  • Grav v1.5.6
  • Admin v1.8.15
@rhukster
Copy link
Member

rhukster commented Feb 1, 2019

Well i've looked into this and it appears to be a bug in the upstream shortcode library. Basically, the open=0 is causing the top parent of the child item to be null. If you use: open="0" it works fine.

It seems to be a problem specifically with the regular parser, as wordpress and regex work fine.

@blueforce
Copy link
Author

no code added? is there a workarround?

@rhukster
Copy link
Member

rhukster commented Feb 4, 2019

The fix was 'downstream' in the thunderer/shortcode library we use:

getgrav/grav-plugin-shortcode-core@f1359f3

@rhukster
Copy link
Member

rhukster commented Feb 4, 2019

also i put workarounds while we were waiting for the updated release of the library here: #29 (comment)

@blueforce
Copy link
Author

thanks a lot, much appricieated

@iamalive-now
Copy link

Hi
I've just started a new site and have added the shortcode ui plugin.
I'm getting much the same error as described above:

"Argument 1 passed to Grav\Plugin\ShortcodeCore\ShortcodeManager::getId() must implement interface Thunder\Shortcode\Shortcode\ShortcodeInterface, null given, called in /home/iamalive/mysite.com/user/plugins/shortcode-ui/classes/shortcodes/AccordionsShortcode.php on line 34 ◀"

I'm using Gantry5 and the Helium Template.

Any ideas?

@rhukster
Copy link
Member

rhukster commented Mar 7, 2021

Are you in the latest version of shortcode core??

@iamalive-now
Copy link

iamalive-now commented Mar 7, 2021 via email

@rhukster
Copy link
Member

rhukster commented Mar 7, 2021

Can you paste your md content?

@iamalive-now
Copy link

iamalive-now commented Mar 7, 2021 via email

@iamalive-now
Copy link

iamalive-now commented Mar 8, 2021 via email

@mahagr
Copy link
Member

mahagr commented Mar 8, 2021

@iamalive-now Please do not send any login credentials into a public discussion (you replied to GitHub notification). I removed them from the GitHub discussion.

@iamalive-now
Copy link

iamalive-now commented Mar 8, 2021 via email

@iamalive-now
Copy link

iamalive-now commented Mar 8, 2021 via email

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

No branches or pull requests

4 participants