By default, SimpleXML returns an array for a blank tag like

<site_information></site_information>

This is not very intuitive, and it should return an empty string. It means we can not use an empty array as a return value from the config system, but I don't know how we would represent an empty array in XML anyways.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

acrollet’s picture

acrollet’s picture

Status: Active » Needs review

setting issue status.

gdd’s picture

Status: Needs review » Needs work

I like this patch, I would just like to see a comment added that the explicitly explains that we have to do this at all is because of SimpleXML's default behavior of returning an array instead of a string.

acrollet’s picture

Status: Needs work » Needs review
FileSize
747 bytes

Comment added, patch attached.

gdd’s picture

Status: Needs review » Reviewed & tested by the community

If this comes back green then its good to go.

Status: Reviewed & tested by the community » Needs work
acrollet’s picture

patch attached with modifications to image to account for new code, tests run fine locally. *fingers crossed*

acrollet’s picture

Status: Needs work » Needs review
gdd’s picture

Status: Needs review » Reviewed & tested by the community

Sweeet

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Great job. :)

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

Anonymous’s picture

Issue summary: View changes

Updated issue summary.