In #1048516: warning: Invalid argument supplied for foreach() in modules/viewfield on line 279. we implemented a workaround for the situation where a user assigns a view to the allowed_views list and later deletes the view. This task is to improve the way viewfield handles this situation.

Comments

sun’s picture

For D7, we should implement proper error handling and possibly maintenance of field values. Options:

  • throwing an error message
  • throwing a watchdog log message
  • validate field values on edit
  • validate field values on render
  • validate allowed values of a field
  • implement Views hooks to react on deletion of views/displays to update field configurations and/or field values accordingly
sun’s picture

jerdavis’s picture

Version: 7.x-2.x-dev » 8.x-3.x-dev
Issue summary: View changes

Updating to 8.x for assessment of where this sits.