Problem/Motivation

Steps to reproduce:

- Install jsonapi + jsonapi_extras
- Navigate to /admin/config/services/jsonapi
- Click on "Overwrite" for the jsonapi_resource_config--jsonapi_resource_config

(or go directly to /admin/config/services/jsonapi/add/jsonapi_resource_config/jsonapi_resource_config)

This happens because ConfigEntityTypeInterface::getPropertiesToExport() can return NULL, and we are assuming on JsonapiResourceConfigForm::buildOverridesForm() Line 235 that it will always be an array.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 After Patch.png48.09 KBakash.addweb
#3 Screen Shot 2018-04-20 at 5.45.24 PM.png212.1 KBakash.addweb
#2 2962694-2.patch1.29 KBmarcoscano

Comments

marcoscano created an issue. See original summary.

marcoscano’s picture

marcoscano
Primary language Portuguese, Brazil
Location Barcelona, Spain
commented
Assigned: marcoscano » Unassigned
Status: Active » Needs review
StatusFileSize
new 2962694-2.patch1.29 KB

This seems to be enough to prevent the warning on the UI.

akash.addweb’s picture

akash.addweb commented
StatusFileSize
new Screen Shot 2018-04-20 at 5.45.24 PM.png212.1 KB
new After Patch.png48.09 KB

@marcoscano, Thanks for resolving this error, this error is now resolved. I checked it using simplytest.me & attached the screenshots for the same.

akash.addweb’s picture

akash.addweb commented
Status: Needs review » Reviewed & tested by the community

  • e0ipso committed 18e5865 on 8.x-2.x authored by marcoscano 
    Issue #2962694 by marcoscano, Ronak.addweb: Warning: array_keys()...
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Status: Reviewed & tested by the community » Fixed

Thanks all! Sorry it took me so long to get here.

Also, welcome to our community @Ronak.addweb!

akash.addweb’s picture

akash.addweb commented

@e0ipso, Thanks for your warm welcome!...

Status: Fixed » Closed (fixed)

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