Port tablesort_example module to Drupal 8

I will work on this.

Thanks a lot!

Check out the examples checklist: #2209627: [meta] Module Checklist for Examples

Attached it is a first shot at tablesort_examples implemented in D8.

Thanks, @jjprieto!

Just a couple things...

If you check out the test result, you'll see that the test for tablesort_example never ran. :-) That's because it's not set up for PSR-4, so the test runner never found the tests.

Also, the menu links need to be defined in a .yml file. Some info here: https://www.drupal.org/node/2228089

Here's a patch with a PSR-4 conversion, and some standards formatting. Also, there was an extra controller called TableSortExample.php, which looked like a copy of the Block controller. I removed that since it was extraneous.

Thanks again.

Thanks @Mile23
I applied your recomendations.

I hope it is better this time

Thanks, @jjprieto. Here's some more. :-)

1. +++ b/modules/examples/tablesort_example/src/Tests/TableSortExampleTest.php
   @@ -0,0 +1,98 @@
   +class TableSortExampleTest extends WebTestBase {
   

   We need another test method for the path defined in *.menu_links.yml.

   Look to phpunit_example for a template on how to do this: http://cgit.drupalcode.org/examples/tree/phpunit_example/src/Tests/PHPUn...

2. +++ b/modules/examples/tablesort_example/tablesort_example.module
   @@ -0,0 +1,34 @@
   +/**
   + * @file
   + * This is an example describing how a module can display data in a sortable
   + * table
   + *
   + * See:
   + * @link http://drupal.org/node/508796 Extenders @endlink
   + * @link http://drupal.org/developing/api/database Database API @endlink
   + */
   +
   +/**
   + * @defgroup tablesort_example Example: Tablesort
   + * @ingroup examples
   + * @{
   + * Example of a sortable table display.
   + */
   

   @file should just say that this file is the module file for tablesort_example.

   The remaining narrative should be in the @defgroup block, and should explain the whats, whys, and hows of the example. Also, the @links should be @sees.

3. +++ b/modules/examples/tablesort_example/tablesort_example.module
   @@ -0,0 +1,34 @@
   +function tablesort_example_help($path, $arg) {
   

   Hook_help() uses route names instead of paths now. https://www.drupal.org/node/2250345

   But none of that matters, because we really should just remove hook_help() and put the information in the examples/tablesort_example page, in the controller. :-)

Ok thanks. I am going to work it

Hi, here I am again jeje.

1.- I changed the tablesort_Example.links.menu.yml (name) and I add the test.
2.- I think I explain the whats, whys, and hows of the example
3.- Deleted hook_help and it added to controller

Thanks a lot for your recomendations. they're very useful

I am going to fix the patch

Second attempt

Sorry but I don't understand. Why did the test fail?

This test it isn't mine Drupal\config_entity_example\Tests\ConfigEntityExampleTest

It isn't at the patch. However my test is ok

Drupal\tablesort_example\Tests\TableSortExampleTest 27 0 0

Yes, the tests fail sometimes, especially for Drupal 8. They keep changing things..

Fixed the branch blocker here: http://cgit.drupalcode.org/examples/commit/?h=8.x-1.x&id=e95cae590b582e5...

Caused by #2305023: Rename getCancelRoute to getCancelUrl

Let's re-test.

Woops... forgot interdiff.

Fixed some formatting issues.

Committed and marked fixed.

Thanks, @jjprieto!