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

Export dialog shows "Report with metadata" even when meta data is not available #12965

Open
tsteur opened this issue May 23, 2018 · 5 comments
Open
Labels
Bug For errors / faults / flaws / inconsistencies etc.

Comments

@tsteur
Copy link
Member

tsteur commented May 23, 2018

image

Selected "with metadata" and then clicked on export. Then got the error "Requested report FormAnalytics.getPageUrls for Website id=1 not found in the list of available reports."

Generated export URL

index.php?apiAction=getPageUrls&apiModule=FormAnalytics&date=2018-05-10&expanded=1&filter_limit=5&format=JSON&idForm=1&idSite=1&method=API.getProcessedReport&module=API&period=month&token_auth=xyz

or maybe it is a problem with the plugin?

@tsteur tsteur added the Bug For errors / faults / flaws / inconsistencies etc. label May 23, 2018
@tsteur tsteur added this to the 3.6.0 milestone May 23, 2018
@sgiehl
Copy link
Member

sgiehl commented Jun 25, 2018

in that specific case it seems to be a problem of the plugin. Will issue a PR there.

But the same occurs when trying to export a goal specific report. Not yet sure how to solve that.

@diosmosis diosmosis modified the milestones: 3.6.0, 3.7.0 Jul 24, 2018
@tsteur
Copy link
Member Author

tsteur commented Sep 18, 2018

I had a quick look...

diff --git a/core/Plugin/Report.php b/core/Plugin/Report.php
index 5356286f80..32113d4c9a 100644
--- a/core/Plugin/Report.php
+++ b/core/Plugin/Report.php
@@ -549,6 +549,13 @@ class Report
         if (!empty($report)) {
             $availableReports[] = $report;
         }
+
+        if ($this->hasGoalMetrics) {
+            $goals = Request::processRequest('Goals.getGoals');
+            foreach ($goals as $goal) {
+                $availableReports[] = $this->buildReportMetadataForGoal($goal);
+            }
+        }
     }

Something like this would be kind of needed that makes the report metadata for all those reports available. Problem is that the "report metadata" would get pretty pretty big and they would be even shown in the mobile app etc which we might not want to... (unless we really do)

Not sure how to fix it otherwise either. It is definitely tricky.

@mattab
Copy link
Member

mattab commented Oct 8, 2018

@tsteur could you maybe open this in a PR so we can see the failing tests and how much data this would be?

@tsteur
Copy link
Member Author

tsteur commented Oct 8, 2018

It's not a solution. That's why I didn't even create the PR

@mattab mattab removed this from the 3.7.0 milestone Oct 11, 2018
@tsteur
Copy link
Member Author

tsteur commented Jun 29, 2021

It's also happening with Users Flow when clicking eg on "interaction 1" and then on export
image

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.
Projects
None yet
Development

No branches or pull requests

5 participants