-
Notifications
You must be signed in to change notification settings - Fork 9
Open
Description
PHP 7.1 / Ubuntu 16.04 / silverstripe/framework 4.3.3
HistoryGridFieldItemRequest
on line 138 expects decorated objects to feature a CMSEditLink()
method, but doesn't enforce / indicate this requirement anywhere.
CMSEditLink()
comes from silverstripe/framework, in the CMSPreviewable
interface, and I would expect decorated objects to throw an exception far earlier in the control chain, if not on install (dev/build). If going with the latter idea, then declaring an add_to_class()
or a constructor on method the DataObjectHistory
extension and checking that the decorated object implements CMSPreviewable
, would be the way to go.
Metadata
Metadata
Assignees
Labels
No labels