REFACTOR: Remove getType() override to allow extensibility

[jsirish]

Summary

Remove the getType() method override so the element inherits BaseElement::getType() which uses i18n_singular_name(). This allows sites to customize the element's display name via extensions by setting $singular_name.

Changes

• Removed the getType() method from ElementCard

Rationale

The base BaseElement::getType() implementation already uses $this->i18n_singular_name():

public function getType()
{
    $default = $this->i18n_singular_name() ?: 'Block';
    return _t(static::class . '.BlockType', $default);
}

By removing the override, extensions can now customize the display name in the CMS element picker by simply setting $singular_name, without needing to override the entire method.

Fixes #20

Related: dynamic/silverstripe-essentials-tools#68

[Copilot]

Pull request overview

This PR removes the getType() method override from ElementCard to improve extensibility. By inheriting BaseElement::getType(), sites can now customize the element's display name through the $singular_name configuration property rather than requiring method overrides.

Key Changes:

• Removed hardcoded getType() method override that returned a static 'Card' translation
• Element now inherits base implementation which uses the existing $singular_name property

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.