Details
-
Type:
Bug
-
Status: Done/Fixed
-
Priority:
Trivial
-
Resolution: Fixed/Completed
-
Affects Version/s: 4.6
-
Fix Version/s: 4.7
-
Component/s: None
-
Labels:None
-
Documentation Required?:None
-
Funding Source:Contributed Code
Description
Make the following changes to the Mailing Summary Report:
1) Change the name of the existing columns to 'Total Opens' and 'Total Open Rate' and make it so the former is no longer a default field in the default instance of the report.
2) Provide two new columns 'Unique Opens' and 'Unique Open Rate', and make the first into a default column in the default instance of the report.
Change the individual mailing report (/civicrm/mailing/report?mid=1263&reset=1) field title of 'Tracked Opens' to 'Unique Opens' and add another field, 'Total Opens' right beneath it.
The mail summary unsubscribe number should be that in the individual mailing report
Attachments
Issue Links
Activity
Submitted PR at https://github.com/civicrm/civicrm-core/pull/6343
Generally the changes are as specified. Exceptions noted are:
1. Warning when clicking Preview on Summary Report instance:
Strict warning: call_user_func() expects parameter 1 to be a valid callback, non-static method CRM_Report_Form_Mailing_Summary::formRule() should not be called statically, assuming $this from compatible context CRM_Report_Form_Mailing_Summary in HTML_QuickForm->validate() (line 1594 of /Users/dgg/git/crm_v4.7/packages/HTML/QuickForm.php).
2. The Columns checked defaults for the default Mailing Summary Report instance in new installs have not been modified as specified in items #1 and #2. Doing so requires modifications to xml/templates/civicrm_navigation.tpl civicrm_report_instance INSERT statement. See last file in this PR for an example: https://github.com/civicrm/civicrm-core/pull/6325/files
3. I'm not sure I understand this statement: "The mail summary unsubscribe number should be that in the individual mailing report."
If it refers to the page at "civicrm/mailing/report?mid=1&reset=1" - then there's still an issue because that page shows Unsubscribe Requests separately from Optout Requests. While the Mailing Summary Report combines those under a single counter for Unsubscribes.
4. (Nice to have .... can ignore ...) Would be nice convenience to link the "Mailing Name" column values to the Mailing Detail Report for that mailing. Currently there's no link to go from summary to detail report.
On #3, should we add a new column or change the name to clarify that it is Unsubscribes plus Opt-outs?
Adding a new column seems clearer. For upgrades, I think it's fine to leave existing instances unchanged (instances which have unsubscribe column selected will potentially have a different count after upgrade since opt-outs will be excluded - but I think that's acceptable).
Pradeep, could you add an Opt-outs field, as well as do #4.
Updated PR to include fix for
1. Strict warning
2. Default column to Unique Open rate during install
3. Added opt out column in Mailing summary report
4. Added link to Mailing name
PR looks good, waiting for PR test to complete.
Pradeep, could you provide an estimate of effort to reroll this against 4.7 head and submit a PR?