Resolve #3230826 "Sorted config"
Closes #3230826
Merge request reports
Activity
added 1 commit
- 1a38d2d9 - Modify `config_test.validation` to allow explicitly testing `orderby` of `type: sequence`.
- Resolved by Wim Leers
293 294 295 /** 296 * {@inheritdoc} 297 */ 298 public function getCanonicalRepresentation(): array { 299 $representation = []; 300 301 // Mapping keys should be ordered however the data definition says. 302 // Note: this specifically does NOT use required/optional keys, because this 303 // MUST work even on invalid data (i.e. when required keys are missing). 304 $ordered_mapping_keys = $this->getValidKeys(); 305 306 // Generate a representation of this mapping: 307 // 1. defer to each value's own canonical representation 308 // (::getElements() respects the stored order in $this->value, not the 309 // schema order) Instead tweaked the comment on
\Drupal\Core\Config\Schema\TypedConfigInterface::getElements()
in 25a33a0c.changed this line in version 16 of the diff
- Resolved by Wim Leers
added 1 commit
- 87bdacaa - Use the same protection against the PHP "resource" type as before.
- Resolved by Wim Leers
- Resolved by Wim Leers
- Resolved by Wim Leers
- Resolved by Wim Leers
22 22 * {@inheritdoc} 23 23 */ 24 24 public function getCastedValue() { 25 if ($this->value === NULL) { 26 return NULL; 27 } 28 - Comment on lines +25 to +28
I thought this was an out-of-scope change and then I realized you're just moving stuff that had previously been done in
StorableConfigBase
. Same thing goes for the other primitives.Edited by Adam G-H changed this line in version 16 of the diff
- Resolved by Wim Leers
- Resolved by Wim Leers
- Resolved by Wim Leers
added 1 commit
- 85af1a96 - The match expression proposed by @phenaproxima is incomplete: `NULL` was not...
added 1 commit
- f2c1390d - Add new `$suppress_exceptions` argument to `::getCanonicalRepresentation()`,...
- Resolved by Wim Leers
added 1 commit
- 60631295 - Update `ViewExecutable::save()` to not rely on array references to internal...
Please register or sign in to reply