AliasWhitelist class references CacheArray in class comment, should reference CacheCollector.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tsphethean’s picture

Status: Active » Needs review
FileSize
641 bytes

Also removed use Drupal\Core\Utility\CacheArray as CacheArray is never used in the AliasWhitelist class.

Berdir’s picture

When referencing the class name, then you must use the full namespace.

I would suggest to reword it to something like "A cache collector to ...", we already have the reference to the class in the extends definition, no need to repeat it.

tsphethean’s picture

Thanks Berdir - makes sense. Re-rolled and attached

Berdir’s picture

Issue summary: View changes
Status: Needs review » Closed (duplicate)

This is being fixed in the RTBC issue #2185015: Remove SchemaCache and CacheArray, closing this as a duplicate.