jhedstrom / DrupalDriver

A collection of drivers for controlling Drupal.
GNU General Public License v2.0
64 stars 95 forks source link

Expose ::isBaseField() on the public interfaces #242

Closed pfrenssen closed 1 year ago

pfrenssen commented 2 years ago

In #114 we introduced a new public Drupal8::isBaseField() method, but this was not exposed on the interfaces since this was Drupal 8 only.

In the meantime Drupal 8+ has become dominant and it would be interesting to expose this method so we can check for base fields.

It would be also interesting to discuss whether we should deal with the distinction between base fields and configurable fields at all. In Drupal 8+ these are handled transparently by the Field API and it would probably make more sense to remove ::isBaseField() and just have ::isField() return TRUE for both base fields and configurable fields. But unfortunately this would break backwards compatibility.