Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Email Report Entry/Exit Pages Sort Order #15176

Open
ibril15 opened this issue Nov 20, 2019 · 8 comments
Open

Email Report Entry/Exit Pages Sort Order #15176

ibril15 opened this issue Nov 20, 2019 · 8 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc. Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement.

Comments

@ibril15
Copy link

ibril15 commented Nov 20, 2019

Hello,

In Email Reports we are able to add Entry Pages and Exit Pages in a report (by URL and by Title). The ordering of these tables is not intuitive. As far as I can tell, the way it works is that it pulls a list of all pages in the time period sorted by number of visits, and then filters out anything that isn't an entry/exit page. That leaves a table with no apparent sort order since there are many frequently popular pages that are not common entry/exit pages.

The tables should be sorted by the data they're displaying: Entrances/Exits descending (in both, the case of displaying the tables by URL and by Title). Can you please confirm this behavior?

Thanks a lot.

image

-Igor

@tsteur tsteur added the c: Usability For issues that let users achieve a defined goal more effectively or efficiently. label Nov 20, 2019
@ibril15
Copy link
Author

ibril15 commented Nov 21, 2019

Hi Thomas,

Thanks for your review. I'm not sure specifically what defines a "usability" issue, but I would actually consider this simply a "bug". It's not that this table is not "user-friendly", it's that it is essentially unusable. It is showing the most popular pages, but not the most popular "entrance pages". So as a result, in one example I have 5 out of the top 7 results with less than 1,000 entrances, while the 8th has almost 115,000. That makes the data unreliable and not presentable.

image

Thanks a lot.

-Igor

@tsteur
Copy link
Member

tsteur commented Nov 21, 2019

Marked it as a bug... @ibril15

Marked it as usability cause in the UI you can change the sorted column but this obviously doesn't work for scheduled report

@tsteur tsteur added Bug For errors / faults / flaws / inconsistencies etc. and removed c: Usability For issues that let users achieve a defined goal more effectively or efficiently. labels Nov 21, 2019
@katebutler katebutler self-assigned this Nov 21, 2019
@tsteur tsteur added this to the 3.13.0 milestone Nov 21, 2019
@mattab mattab modified the milestones: 3.13.0, 3.13.1 Nov 27, 2019
@mattab mattab removed this from the 3.13.1 milestone Jan 8, 2020
@utrautmann
Copy link

@tsteur A customer of mine has the same problem and cannot work meaningfully with the unsorted output in the reports.
Is there an easy way to have the reports sort this list? Sounds like an order statement in SQL?

@tsteur
Copy link
Member

tsteur commented Jun 7, 2021

Hi @utrautmann it's not a SQL statement issue but it would likely require sorting after filtering actions. Something like the diff below but please note that's just more of a guess I didn't actually test it.

--- a/plugins/Actions/API.php
+++ b/plugins/Actions/API.php
@@ -170,6 +170,7 @@ class API extends \Piwik\Plugin\API
 
         $dataTable = $this->getPageUrls($idSite, $period, $date, $segment, $expanded, $idSubtable, false, $flat);
         $this->filterNonEntryActions($dataTable);
+        $dataTable->filter('Sort', ['entry_nb_visits']);
         return $dataTable;
     }
 
@@ -183,6 +184,7 @@ class API extends \Piwik\Plugin\API
 
         $dataTable = $this->getPageUrls($idSite, $period, $date, $segment, $expanded, $idSubtable, false, $flat);
         $this->filterNonExitActions($dataTable);
+        $dataTable->filter('Sort', ['exit_nb_visits']);
         return $dataTable;
     }
 
@@ -219,6 +221,7 @@ class API extends \Piwik\Plugin\API
 
         $dataTable = $this->getPageTitles($idSite, $period, $date, $segment, $expanded, $idSubtable, $flat);
         $this->filterNonEntryActions($dataTable);
+        $dataTable->filter('Sort', ['entry_nb_visits']);
         return $dataTable;
     }
 
@@ -233,6 +236,7 @@ class API extends \Piwik\Plugin\API
 
         $dataTable = $this->getPageTitles($idSite, $period, $date, $segment, $expanded, $idSubtable, $flat);
         $this->filterNonExitActions($dataTable);
+        $dataTable->filter('Sort', ['exit_nb_visits']);
         return $dataTable;
     }

@utrautmann
Copy link

@tsteur Thank you. Unfortunately, it doesn't help. I patched the API.php but the sort order remains unsorted.

@tsteur
Copy link
Member

tsteur commented Jul 18, 2021

Thanks for letting us know @utrautmann Looks like it also needs some other changes somewhere.

@tsteur tsteur added the Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement. label Jul 18, 2021
@utrautmann
Copy link

I want to ask for the status of this bug issue. Another customer mentioned this problem.

@Stan-vw
Copy link
Contributor

Stan-vw commented Jun 15, 2023

We've got it on our backlog. It's not currently at the top but we've discussed it with the design team a few times, also while having a look at other feature requests that relate to how tables work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc. Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement.
Projects
None yet
Development

No branches or pull requests

7 participants