Skip to content

Commit

Permalink
Merge pull request #3268 from craftcms/fix/3263-qualify-email-query-o…
Browse files Browse the repository at this point in the history
…rder-by-v3

Fix/3263 qualify email query order by v3
  • Loading branch information
lukeholder authored Sep 7, 2023
2 parents fc08a7b + 68aa538 commit 00fd913
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Release Notes for Craft Commerce

## Unreleased

- Fixed a bug where emails were not being ordered by name correctly. ([#3263](https://github.com/craftcms/commerce/issues/3263))

## 3.4.22.1 - 2023-06-03

- Fixed a bug where the incorrect version number was being shown after updating.
Expand Down
2 changes: 1 addition & 1 deletion src/services/Emails.php
Original file line number Diff line number Diff line change
Expand Up @@ -910,7 +910,7 @@ private function _createEmailQuery(): Query
'emails.plainTextTemplatePath',
'emails.uid',
])
->orderBy('name')
->orderBy('emails.name')
->from([Table::EMAILS . ' emails']);

// todo: remove schema version condition after next beakpoint
Expand Down

0 comments on commit 00fd913

Please sign in to comment.