CiviCRM Community Forums (archive)

*

News:

Have a question about CiviCRM?
Get it answered quickly at the new
CiviCRM Stack Exchange Q+A site

This forum was archived on 25 November 2017. Learn more.
How to get involved.
What to do if you think you've found a bug.



  • CiviCRM Community Forums (archive) »
  • Old sections (read-only, deprecated) »
  • Support »
  • Using CiviCRM »
  • Using CiviReport (Moderator: Dave Greenberg) »
  • "Illegal offset type" warning for Extended Reports
Pages: 1 [2] 3 4

Author Topic: "Illegal offset type" warning for Extended Reports  (Read 5161 times)

Eileen

  • Forum Godess / God
  • I’m (like) Lobo ;)
  • *****
  • Posts: 4195
  • Karma: 218
    • Fuzion
Re: "Illegal offset type" warning for Extended Reports
September 17, 2013, 04:51:05 pm
Hmm serves me right for doing the fixes on a sample-data version of civi. It was-error free on that because of the limited data & I couldn't really test for the same reason. I think I can see the error - will check on a customer site

Make today the day you step up to support CiviCRM and all the amazing organisations that are using it to improve our world - http://civicrm.org/contribute

Eileen

  • Forum Godess / God
  • I’m (like) Lobo ;)
  • *****
  • Posts: 4195
  • Karma: 218
    • Fuzion
Re: "Illegal offset type" warning for Extended Reports
September 17, 2013, 08:31:27 pm
OK - I did a few more fixes on this. Some of the code needs a bit or a re-write - but try now.

Also, I added in assignee contact (before it was only target & source).

If this fixes & relationship extended report isn't broken in the process I'll publish a new version since all of sudden these 2 reports seem to be in demand
Make today the day you step up to support CiviCRM and all the amazing organisations that are using it to improve our world - http://civicrm.org/contribute

JohnM

  • I post occasionally
  • **
  • Posts: 36
  • Karma: 0
  • CiviCRM version: 4.4.6
  • CMS version: Drupal 7
Re: "Illegal offset type" warning for Extended Reports
September 18, 2013, 06:02:12 am
Thanks Eileen - Reports now load correctly when the Create Reports from Template option is selected. (The previous error message has now gone completely). However clicking the Preview button now shows error messages on 3 of the reports. (See below). Most of the reports also have filter issues throwing ‘Sorry but we are not able to provide this at the moment message’. This is mainly to do with custom fields being used for sorting although I did find one report that could not filter on Groups.

This is what I found when I updated this morning…

1. Extended Reports (Price - Contribution Based) – Working OK with custom fields. Filter issues.

2. Extended Reports (Price Line Item Membership) – Working OK with custom fields. Filter issues.

3. Extended Reports (Price Line Item) – Working OK but NO custom fields useable. Filter issues.

4. Extended Reports (Price Lline Item Participant) – Working OK with custom fields. Filter issues.

Reports with errors:

5. Basic contact report (Multiple Phone Numbers) – Working OK with Custom fields BUT ERRORS displayed:

Warning: array_key_exists() expects parameter 2 to be array, null given in CRM_Extendedreport_Form_Report_ExtendedReport->selectableCustomDataFrom() (line 1442 of /home/admin/public_html/sites/all/civicrm_extensions/nz.co.fuzion.extendedreport-1.4/nz.co.fuzion.extendedreport-1.4/CRM/Extendedreport/Form/Report/ExtendedReport.php).

6. Activity Extended – Working OK with Custom fields BUT ERRORS displayed:

Warning: array_key_exists() expects parameter 2 to be array, null given in CRM_Extendedreport_Form_Report_ExtendedReport->selectableCustomDataFrom() (line 1442 of /home/admin/public_html/sites/all/civicrm_extensions/nz.co.fuzion.extendedreport-1.4/nz.co.fuzion.extendedreport-1.4/CRM/Extendedreport/Form/Report/ExtendedReport.php).

7. Extended Relationship Report - Working OK with Custom fields BUT ERRORS displayed:

Warning: array_key_exists() expects parameter 2 to be array, null given in CRM_Extendedreport_Form_Report_ExtendedReport->selectableCustomDataFrom() (line 1442 of /home/admin/public_html/sites/all/civicrm_extensions/nz.co.fuzion.extendedreport-1.4/nz.co.fuzion.extendedreport-1.4/CRM/Extendedreport/Form/Report/ExtendedReport.php).

Reports 6-8 all have Filter issues.

Thanks for the quick work on trying to resolve this. I hope the above is of some help.

Eileen

  • Forum Godess / God
  • I’m (like) Lobo ;)
  • *****
  • Posts: 4195
  • Karma: 218
    • Fuzion
Re: "Illegal offset type" warning for Extended Reports
September 18, 2013, 07:58:50 pm
OK - try now.

I can see the problems are occuring because I tried to minimise the number of methods I over-road - but it resulted in difficult code.
Make today the day you step up to support CiviCRM and all the amazing organisations that are using it to improve our world - http://civicrm.org/contribute

JohnM

  • I post occasionally
  • **
  • Posts: 36
  • Karma: 0
  • CiviCRM version: 4.4.6
  • CMS version: Drupal 7
Re: "Illegal offset type" warning for Extended Reports
September 19, 2013, 02:32:00 am
Thanks Eileen. This is what I seeing now…

1. Extended Reports (Price - Contribution Based) – Working OK with custom fields and filters.

2. Extended Reports (Price Line Item) – Working OK with custom fields and filters.

3. Extended Reports (Price Line Item Participant) – Working OK with custom fields and filters.

4. Extended Reports (Price Line Item Membership) – Working OK with custom fields and filters.

Reports with errors:

5. Basic contact report (Multiple Phone Numbers) – Working OK with Custom fields and filters.
Error message displayed
Warning: array_key_exists() expects parameter 2 to be array, null given in CRM_Extendedreport_Form_Report_ExtendedReport->selectableCustomDataFrom() (line 1459 of /home/admin/public_html/sites/all/civicrm_extensions/nz.co.fuzion.extendedreport-1.4/CRM/Extendedreport/Form/Report/ExtendedReport.php).

6. Activity Extended – Working OK with Custom fields and filters.
2 errors displayed:
Warning: array_key_exists() expects parameter 2 to be array, null given in CRM_Extendedreport_Form_Report_ExtendedReport->selectableCustomDataFrom() (line 1459 of /home/admin/public_html/sites/all/civicrm_extensions/nz.co.fuzion.extendedreport-1.4/CRM/Extendedreport/Form/Report/ExtendedReport.php).

Message from Webpage “showBlocks not in.tpl = option field 6”

7. Extended Relationship Report - Working OK with custom fields and filters.
Error message displayed
Warning: array_key_exists() expects parameter 2 to be array, null given in CRM_Extendedreport_Form_Report_ExtendedReport->selectableCustomDataFrom() (line 1459 of /home/admin/public_html/sites/all/civicrm_extensions/nz.co.fuzion.extendedreport-1.4/CRM/Extendedreport/Form/Report/ExtendedReport.php).

There are also some fields that need checking (e.g. There are 2 Contact B ::Nick Name fields. One of these should be Contact A: Nick Name, etc)

Eileen

  • Forum Godess / God
  • I’m (like) Lobo ;)
  • *****
  • Posts: 4195
  • Karma: 218
    • Fuzion
Re: "Illegal offset type" warning for Extended Reports
September 19, 2013, 05:31:26 am
ok - I've realised why you get an error & I don't & pushed a fix on that
Make today the day you step up to support CiviCRM and all the amazing organisations that are using it to improve our world - http://civicrm.org/contribute

JohnM

  • I post occasionally
  • **
  • Posts: 36
  • Karma: 0
  • CiviCRM version: 4.4.6
  • CMS version: Drupal 7
Re: "Illegal offset type" warning for Extended Reports
September 19, 2013, 10:17:13 am
Hi Eileen. That seems to have done it - warning messages have now disappeared.

Basic contact report (Multiple Phone Numbers)
- Working fine now

Activity Extended 
- Still has a message from Webpage about “showBlocks not in.tpl = option field 6”

Extended Relationship Report 
- Working fine now

I would like to use the report fairly soon. Would it be possible to tidy the following…

1. Contact B: Nick Name appears 2xs. Looks like this needs correcting to Contact A: Nick Name.

2. Contact B: First Name appears 2xs. Looks like this needs correcting to A: First Name.

3. Contact B:Middle Name appears 2xs. Looks like this needs correcting to A: Middle Name.

4. Contact B:Last Name appears 2xs. Looks like this needs correcting to A: Last Name.

5. Sorting – at the moment sorting is set 2x on Contact B: Name. Sorting on Contact A: Name is really needed

6. Set Filters – these are repeated 2xs on the form – looks like the code has been double pasted?

6. It would also be helpful if the Contact Names had a link in them so they could be connected to the record cards underneath

A couple of other thoughts for the future, would it be possible to have sorting included in all of the extended reports? Also, is there any way membership information could be exposed in the Enhanced Relationship Report? I realise that this is a tall order but again I think this could be a useful feature.

Thanks for all of the great work you are doing with this extension. It really helps strengthen the reporting features of CiviCRM.

Eileen

  • Forum Godess / God
  • I’m (like) Lobo ;)
  • *****
  • Posts: 4195
  • Karma: 218
    • Fuzion
Re: "Illegal offset type" warning for Extended Reports
September 19, 2013, 10:40:33 pm
I've fixed up that double-up bug. The contact id field actually has the links on it at the moment - rather than the display_name field.

There are actually a few other reports in that extension - pledge sybunt etc, the pivot one (which one works in quite specific circumstances & the event editing grid)
Make today the day you step up to support CiviCRM and all the amazing organisations that are using it to improve our world - http://civicrm.org/contribute

JohnM

  • I post occasionally
  • **
  • Posts: 36
  • Karma: 0
  • CiviCRM version: 4.4.6
  • CMS version: Drupal 7
Re: "Illegal offset type" warning for Extended Reports
September 20, 2013, 05:51:00 am
Thanks for sorting this Eileen.

I am still finding problems with there being no 'Contact A: Name' in the Order by Columns list. Looks like this is missing from the select list.

Under Set Filters there is also some confusion with the labelling of the Custom fields. At the moment there is no A or B referencing the field set options for the 2 types of contact.

If these can be sorted I think we will have a fully working report. Thanks for taking the trouble to do this so quickly - it is very much appreciated!

JohnM

  • I post occasionally
  • **
  • Posts: 36
  • Karma: 0
  • CiviCRM version: 4.4.6
  • CMS version: Drupal 7
Re: "Illegal offset type" warning for Extended Reports
September 20, 2013, 06:08:54 am
Just found something else. It looks like Contact A: and B: Nick Names are not working - possibly only picking up Contact B data? In the report both Contact A and B are blank although we definitely have data that should be in the Contact A field.

I have also just filtered on one of the custom fields and it is showing the same data for both contacts. Looks very much like the custom field sets have been duplicated rather then being set up for Contact A and Contact B use. Hope that all makes sense!

liblogger

  • I’m new here
  • *
  • Posts: 19
  • Karma: 0
  • CiviCRM version: 4.3.5
  • CMS version: Drupal 7.23
  • MySQL version: 5.0.7
  • PHP version: 5.3.5
Re: "Illegal offset type" warning for Extended Reports
September 20, 2013, 08:32:44 am
Hi Eileen - thanks for all the work you've been doing on this.

With the latest version, I'm not getting any error messages on the copy of our production site, but none of the custom data fields are available as filters. I can add the custom data in columns, but not filter by them.

On my fresh installation, I get the following notice:

Notice: Undefined index: filters in CRM_Extendedreport_Form_Report_ExtendedReport->addSelectableCustomFields() (line 1517 of C:\xampp\htdocs\botch\sites\default\files\civicrm\custom_ext\nz.co.fuzion.extendedreport-master\CRM\Extendedreport\Form\Report\ExtendedReport.php).

And as with the other site, it's not giving me the custom data as a filter.

The other main issue I'm noticing is that the 'Income Count Summary Report' isn't displaying event participant counts, but this might be because we haven't been recording event fees - I've not had a chance to test the other reports beyond checking that they're not throwing out errors, nor have I had a chance to test it on the fresh installation with event fees included. I'll try and get some time next week to test this further.

Eileen

  • Forum Godess / God
  • I’m (like) Lobo ;)
  • *****
  • Posts: 4195
  • Karma: 218
    • Fuzion
Re: "Illegal offset type" warning for Extended Reports
September 20, 2013, 02:57:30 pm
A couple more fixes done. Liblogger are using the latest from github?
Make today the day you step up to support CiviCRM and all the amazing organisations that are using it to improve our world - http://civicrm.org/contribute

JohnM

  • I post occasionally
  • **
  • Posts: 36
  • Karma: 0
  • CiviCRM version: 4.4.6
  • CMS version: Drupal 7
Re: "Illegal offset type" warning for Extended Reports
September 21, 2013, 08:58:59 am
Thanks Eileen. Looking at the reports I am finding that the 'Nick Name' field is not working correctly in 2 of the reports:

1. Enhanced Relationship Report
2. Extended Activity Report - includes multi-contact custom fields[/li][/list]

All of the others I've looked at seem fine.

I am also finding that the filtering on the Enhance Report is still duplicating. This is what is appearing at the top of the report:

Record Status
Is one of ACTIVE, PENDING, ACTIVE, PENDING

Record Status
Is one of ACTIVE, PENDING, ACTIVE, PENDING

Filtering on custom fields seems to work fine but does not give you the opportunity to filter on A or B contact - only one duplicate option.

I have also found that it is necessary to flush all catches to get the updated reports to work correctly.

One thing that is missing at the moment is the ability to filter by SubType. In our case we are using Students, Parents and Tutors sub types - this would be a really helpful addition to the report(s) if it can be added.

A thought for the final release - There are a lot of great reports in this extension. At the moment they are getting mixed up with standard Civi reports and custom reports. It would be helpful if they all had the words "Extended Reports" added to both Report Title and Report Descriptions. This would make them much easier to find.

liblogger

  • I’m new here
  • *
  • Posts: 19
  • Karma: 0
  • CiviCRM version: 4.3.5
  • CMS version: Drupal 7.23
  • MySQL version: 5.0.7
  • PHP version: 5.3.5
Re: "Illegal offset type" warning for Extended Reports
September 23, 2013, 03:58:59 am
Hi Eileen - yes, as far as I know it's the latest version. I downloaded the 'master' zip file on Friday 20th, and I don't think there have been any changes made since the 19th?

Eileen

  • Forum Godess / God
  • I’m (like) Lobo ;)
  • *****
  • Posts: 4195
  • Karma: 218
    • Fuzion
Re: "Illegal offset type" warning for Extended Reports
September 23, 2013, 04:11:01 pm
Liblogger made me realise I hadn't actually pushed the latest changes to github - they are there now. Hopefully not

NB - I added the Extended report to the titles per your suggestion JohnM.



Make today the day you step up to support CiviCRM and all the amazing organisations that are using it to improve our world - http://civicrm.org/contribute

Pages: 1 [2] 3 4
  • CiviCRM Community Forums (archive) »
  • Old sections (read-only, deprecated) »
  • Support »
  • Using CiviCRM »
  • Using CiviReport (Moderator: Dave Greenberg) »
  • "Illegal offset type" warning for Extended Reports

This forum was archived on 2017-11-26.