- Issue created by @BenStallings
- First commit to issue fork.
- 🇮🇳India kalpanajaiswal
Created a MR
MR: https://git.drupalcode.org/project/drupal/-/merge_requests/8325/** * The fields on which to sort. * * @var array */ protected $fields; /** * Constructs a new Sort object. * * Takes an array of sort fields. Example: * [ * [ * 'path' => 'changed', * 'direction' => 'DESC', * ], * [ * 'path' => 'title', * 'direction' => 'ASC', * 'langcode' => 'en-US', * ], * ] * * @param array $fields * The entity query sort fields. */ public function __construct(array $fields) { $this->fields = $fields; }
- Status changed to Needs review
6 months ago 9:47am 10 June 2024 - Status changed to Needs work
6 months ago 1:11pm 10 June 2024 - 🇺🇸United States smustgrave
Seems straightforward but MR should be against 11.x
- First commit to issue fork.
- 🇮🇳India kalpanajaiswal
MR https://git.drupalcode.org/project/drupal/-/merge_requests/8325 has been updated against 11.x
- Status changed to Needs review
5 months ago 4:15am 25 June 2024 - 🇮🇳India ankitv18
ankitv18 → changed the visibility of the branch 3453068-property-type-update to hidden.
- 🇮🇳India ankitv18
ankitv18 → changed the visibility of the branch 3453068-sortfields-property-is to hidden.
- Status changed to RTBC
5 months ago 1:09pm 25 June 2024 - 🇮🇳India ankitv18
MR!8367 change looking as expected as per proposed solution.
- 🇳🇱Netherlands bbrala Netherlands
Yes, i think this is fine like this since it is only as docblock.
-
longwave →
committed 35848a5e on 10.3.x
Issue #3453068 by kalpanajaiswal, BenStallings: Sort::fields property is...
-
longwave →
committed 35848a5e on 10.3.x
- 🇬🇧United Kingdom longwave UK
Crediting @BenStallings for the original report and @kalpanajaiswal for the fix.
Backported down to 10.3.x as a docs-only fix.
Committed and pushed 9f6143c129 to 11.x and 6f54f45dc2 to 11.0.x and d5deca47b1 to 10.4.x and 35848a5ec5 to 10.3.x. Thanks!
-
longwave →
committed d5deca47 on 10.4.x
Issue #3453068 by kalpanajaiswal, BenStallings: Sort::fields property is...
-
longwave →
committed d5deca47 on 10.4.x
-
longwave →
committed 6f54f45d on 11.0.x
Issue #3453068 by kalpanajaiswal, BenStallings: Sort::fields property is...
-
longwave →
committed 6f54f45d on 11.0.x
-
longwave →
committed 9f6143c1 on 11.x
Issue #3453068 by kalpanajaiswal, BenStallings: Sort::fields property is...
-
longwave →
committed 9f6143c1 on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.