Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 295813

Remove public properties from entity classes

$
0
0

I don't know how the EntityNG conversions factor in to all of this, but as far as ConfigEntity goes, all of the "entity keys" (id/label/bundle/uuid) are public properties. They shouldn't be. We should work toward making them all protected.

This is just a first attempt. To see what happens.

AttachmentSizeStatusTest resultOperations
protected-properties-ftw.patch16.97 KBIdleFAILED: [[SimpleTest]]: [MySQL] Drupal installation failed.View details | Re-test

Viewing all articles
Browse latest Browse all 295813

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>