This repository was archived by the owner on Nov 9, 2026. It is now read-only.
Repository navigation
Release 2.0.0 - #1
Merged
Merged
Conversation
- composer: php ^8.1, framework ^5 || ^6, elemental ^5.4 || ^6, admintweaks ^3 || ^4, vendor-plugin ^2 || ^3; suggest elemental-virtual and subsites; funding entry. - Extensions extend Extension instead of DataExtension (removed in 6), without the parent::onBeforeWrite() calls that only DataExtension could answer. - Block descriptions use class_description / i18n_classDescription(): elemental 5.4 ignores the old 'description' config and elemental 6 removed getDescription(). - BlockAdmin links the page through getCMSEditLink() where it exists. - BlockIconsPreviewController: index() now matches LeftAndMain::index(), which fataled every flush on Silverstripe 5 and 6; it lists BaseElement subclasses (Block::class never resolved); the inline template moved to a .ss file (no SSViewer::fromString() in 6); the ArrayList class is picked per major.
…ecode(null)
- BlockContent::FieldEnabled('Content') answered has_image and ('Image') answered
has_content.
- ElementContentMigrationExtension took $migratable and $skip by value, so its
assignments never reached MigrateContentToElement.
- getExtraData() passed NULL to json_decode() for a block without ExtraData, a
deprecation since PHP 8.1.
Silverstripe 6 types the argument as array, so opening a block in the Content Blocks admin threw a TypeError. An array works on Silverstripe 5 too.
Covers BlockContent (schema, has_* config, upload folder, ExtraData, templates), ElementBlockBaseExtension, ContentBlocksToggleExtension, the block admin, the block-type icon preview, the migration hooks, the module config, and the optional elemental-virtual and subsites integrations (skipped when those are not installed). Each defect fixed in this pass has a regression test.
A mutation that returned getType() survived: for BlockBase subclasses getType() is the class description already.
… boot step Matrix: Silverstripe 5 on PHP 8.1 and 8.3, Silverstripe 6 on PHP 8.3 and 8.4, with elemental-virtual and subsites installed. Tests run before the database is built, with --fail-on-empty-test-suite on every leg. .gitattributes keeps tests and CI files out of dist installs.
README: requirements, compatibility table, what installing configures, every config option and public method, the optional integrations, running the tests, sponsorship line.
…de dev As a LeftAndMain with a url_segment it was listed in the CMS menu (titled with its class name) and routed at admin/admin/blocktypeicons in every environment; only the Director rule was dev-gated. It now sets ignore_menuitem and answers 404 from init() outside dev.
… stop add(null) throwing LinkedElementRelation() called add() with the clone's LinkedElementID even when a new clone had none, which throws InvalidArgumentException. The onBeforeWrite() transfer was not covered by any test.
…ement's class_description rarely shows
…v-only scope, the v1 branch; ship UPGRADING.md
…lementRelation fix in CHANGELOG
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See CHANGELOG.md.