Since a few of us have volunteered to help with the testing, I am opening a testing thread for
6.x-3.x-dev
to share back observations and issues. Bare with me if I am not doing this the right way.

This concerns the version put online on (about) Oct 15 (not the one from Oct22).

1. Overview table

- Showing on one instance, empty on another
- Details link points to Overview table because the node number is missing at the end of the link
- Node number is also missing under the respective newsletter node links in the first column of the table

2. Opens table

I get this error message, while table remains empty.

warning: pg_query() [function.pg-query]: Query failed: ERROR: column "simplenews_statistics_opens.timestamp" must appear in the GROUP BY clause or be used in an aggregate function in /var/www/includes/database.pgsql.inc on line 138.
user warning: query: select email, timestamp, count(email) as opens from simplenews_statistics_opens where nid=18 group by email in /var/www/sites/all/modules/contrib/simplenews_statistics/simplenews_statistics.module on line 273.
warning: pg_query() [function.pg-query]: Query failed: ERROR: column "simplenews_statistics_opens.timestamp" must appear in the GROUP BY clause or be used in an aggregate function in /var/www/includes/database.pgsql.inc on line 138.
user warning: query: select email, timestamp, count(email) as opens from simplenews_statistics_opens where nid=18 group by email ORDER BY timestamp DESC LIMIT 50 OFFSET 0 in /var/www/sites/all/modules/contrib/simplenews_statistics/simplenews_statistics.module on line 273.

3. Clicks table

Looks fine.

I manually added the /NODE# and /NODE#/clicks extensions to get to the Details tables.

Please advise if I should report back in a different way.

Comments

weseze’s picture

weseze commented

Great idea, love this! But I would love it even more if you could find the time to test these issues against the latest dev.
Nevertheless I'll certainly be looking into these issues. Most likely in the first week of november, haven't got much time right now ;)

rmcom’s picture

rmcom commented

Great, I will test the new version asap but can't use my dev instance at the moment. I will be able to use it again by mid-week.

If you like this thread, folks, please post your test results and observations here, so that we can help weseze as much as possible with the great work he is doing!

Many thanks for the simplenews_analytics integration! I am looking forward to testing this in the latest dev version.

weseze’s picture

weseze commented

number 1: is fixed and comitted to 3.x-dev
number 2: can't find that one, could you test against latest dev?
number 3: I don't understand...

I think it might be a good idea to post seperate issues for every individual problem and keep this issue as a sort of "overview issue" for 3.x-dev testing.

rmcom’s picture

rmcom commented

1. great! thank you!
2. will do and post back here
3. meant: the links from the overview table to clicks were not working, but by typing in admin/content/simplenews/statistics/132/clicks (132 = NODE# placeholder from above) I was able to get to the clicks table.

Thank you and will post back test results here!

weseze’s picture

weseze commented
Status: Active » Closed (fixed)

Maybe it's better to seperate the issues. Makes it easier to follow-up, commit patches, ...