Order activity log view (commerce_activity) has default sort set to Log ID, but table settings has default sort set to date (created).
It causes problems when the log entries are recorded in a short time interval (in the same second), e.g. if order transition state is forwarded to a next state in the event subscribers and several log entries are saved at once.

To solve this small problem, we need to set default sort for table to None.

CommentFileSizeAuthor
#4 fix_activity_log_sort_3013720_3.patch361 bytesmartin_klima
#2 fix_activity_log_sort_3013720_2.patch357 bytesmartin_klima
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

martin_klima created an issue. See original summary.

martin_klima’s picture

martin_klima
he/his
Primary language Czech
CreditAttribution: martin_klima at Czech Drupal Association for HBF s.r.o. commented
Status: Active » Needs review
FileSize
fix_activity_log_sort_3013720_2.patch357 bytes

Status: Needs review » Needs work

The last submitted patch, 2: fix_activity_log_sort_3013720_2.patch, failed testing. View results

martin_klima’s picture

martin_klima
he/his
Primary language Czech
CreditAttribution: martin_klima at Czech Drupal Association for HBF s.r.o. commented
Status: Needs work » Needs review
FileSize
fix_activity_log_sort_3013720_3.patch361 bytes

Sorry, I did the patch manually in Phpstorm. Second chance.

martin_klima’s picture

martin_klima
he/his
Primary language Czech
CreditAttribution: martin_klima at Czech Drupal Association for HBF s.r.o. commented
Siegrist’s picture

Siegrist
Primary language German
CreditAttribution: Siegrist at Liip commented
Status: Needs review » Reviewed & tested by the community

Stumbled across this problem and #4 fixes the order perfectly! Thanks.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks, not reverting the view in case customizations were made but at least it'll be fixed on new installs :p.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.