Skip to content

FieldsetClose - UIkit Admin Theme Error #571

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

Closed
chriswthomson opened this issue Apr 23, 2018 · 2 comments
Closed

FieldsetClose - UIkit Admin Theme Error #571

chriswthomson opened this issue Apr 23, 2018 · 2 comments

Comments

@chriswthomson
Copy link

chriswthomson commented Apr 23, 2018

Short description of the issue

When attempting to edit a FieldsetClose field on 3.0.99, an error is encountered:

Expected behavior

Field editor is displayed

Actual behavior

Error: Uncaught TypeError: Argument 2 passed to ProcessWire\AdminThemeUikitConfigHelper::configInputfield() must be an instance of ProcessWire\InputfieldWrapper, null given, called in .../wire/modules/AdminTheme/AdminThemeUikit/AdminThemeUikit.module on line 450 and defined in .../wire/modules/AdminTheme/AdminThemeUikit/config.php:362
Stack trace:
#0 .../wire/modules/AdminTheme/AdminThemeUikit/AdminThemeUikit.module(450): ProcessWire\AdminThemeUikitConfigHelper->configInputfield(Object(ProcessWire\InputfieldFieldsetClose), NULL)
#1 .../wire/core/WireHooks.php(822): ProcessWire\AdminThemeUikit->hookAfterInputfieldGetConfigInputfields(Object(ProcessWire\HookEvent))
#2 .../wire/core/Wire.php(442): ProcessWire\WireHooks->runHooks(Object(ProcessWire\InputfieldFieldsetClose), 'getConfigInputf...', Array)
#3 .../wire/core/Field.php(1013): ProcessWire\Wire->__call('getConfigInputf...', Array)
#4 /var/www/processwire/w (line 362 of .../wire/modules/AdminTheme/AdminThemeUikit/config.php) 

This error message was shown because: you are logged in as a Superuser. Error has been logged.

Steps to reproduce the issue

  1. Create and save a new Fieldset in Tab (Open) field
  2. Try to edit the automatically created "_END" field (FieldsetClose)

Setup/Environment

  • ProcessWire version: 3.0.99
  • AdminThemeUikit 0.2.7

I'm not getting this on 3.0.98 Master, nor if I use the default PW theme.

Cheers,

Chris

ryancramerdesign added a commit to processwire/processwire that referenced this issue Apr 25, 2018
…dtypeFieldsetClose in Setup > Fields would result in error in AdminThemeUikit
@ryancramerdesign
Copy link
Member

Thanks, I've pushed a fix for this issue.

@netcarver
Copy link
Collaborator

@chriswthomson Hello again Chris. I see that Ryan pushed a fix for this issue. If that resolved things to your satisfaction, please could you close this issue, otherwise could you add a reply here so we can keep it on Ryan's radar. Thank you!

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

No branches or pull requests

3 participants