Entity names in the 'add view' form are unsorted

@joachim,

Thanks for your contribution. Given patch is working as expected to me. Please refer the attached image for more reference.

Thank you.

If this is something we care about, we need a test

Thanks

Fixing this makes sense to me, but the fix in 11 still doesn't look right because we are sorting on objects I suspect. Casting the titles to strings makes it look good.

Expanded the IS, added some screenshots and some title clean up.

Still needs tests.

Also not 100% convinced this is the best we can do. Using the example of commerce, it might be nice to group entities by package or something. But maybe alphabetically is a good first step...

Here is a test for this. Test only file is the interdiff.

HI Lendude I have applied your patch successfully it's showing sorted content now

@joachim

The asort() function sorts an associative array in ascending order, according to the value
AND
The natsort() function sorts an array by using a "natural order" algorithm. The values keep their original keys.

example :- In a natural algorithm, the number 2 is less than the number 10. In computer sorting, 10 is less than 2, because the first number in "10" is less than 2.

@joachim applied patch for same. thanks

@chanderbhushan thanks! but can we get the test coverage in the patch too?

+++ b/core/modules/views_ui/tests/src/Functional/WizardTest.php
@@ -64,4 +64,103 @@ public function testWizardFieldLength() {
+    \Drupal::service('module_installer')->install(['taxonomy']);
...
+    // Installing comment module.
+    \Drupal::service('module_installer')->install(['comment']);
...
+    \Drupal::service('module_installer')->install(['file']);
+    \Drupal::service('module_installer')->install(['block_content']);

Let's install all these modules in one go and then do one assert. The piecemeal install doesn't really prove too much. Note that ->install() takes a list of all modules to install so it's only necessary to have a single call.

Thanks to @alexpott for reviewing the patch.

Tested via above mentioned steps and latest patch. This can be moved to RTBC

I did not find this issue when I created #3183106: Sort the options in the "Add view" wizard. One of these issues should be closed as a duplicate.

Even though this issue is older, I suggest closing it since the approach used on the other issue is simpler. As I said in #3183106-1: Sort the options in the "Add view" wizard,

I believe that the test coverage in #3065903: Add label sort ability to Select element is good enough that we can just use the new sort_options attribute here without adding any new tests.

If I remember correctly, a lot of effort went into #3065903: Add label sort ability to Select element to make sure that the sort is done at the correct time. I do not think there are any variable substitutions in this use case, but has anyone tested the patch here with translations?

If no one objects, then I will close this issue as a duplicate and transfer the issue credits to #3183106.

Closing in favour of the other issue. I've transferred issue credit.