I tried to specify property, but did not realize that it would crash PHP completely if returned value is not string (when running str_replace() with value.) Attached patch adds check for presence of string, or otherwise throws catch-able error with details on returned value, so user can go back and fix content.

CommentFileSizeAuthor
add-property-value-check-and-better-error-handling.patch1.44 KBpavel.karoukin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

  • sdrycroft committed 84a61e6 on 
    Issue #2527012. Ensure that the value is a string.
sdrycroft’s picture

sdrycroft CreditAttribution: sdrycroft commented
Assigned: Unassigned » sdrycroft
Status: Active » Fixed

Thanks Pavel, I had already fixed this in my working repository, but hadn't thought about releasing it here.

https://github.com/NaturalHistoryMuseum/scratchpads2/tree/master/sites/a...

Status: Fixed » Closed (fixed)

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