eileenmcnaughton / nz.co.fuzion.csvimport

Import to API using csv GUI
GNU Affero General Public License v3.0
8 stars 27 forks source link

Missing Entity select field #71

Closed savionlee closed 2 months ago

savionlee commented 2 months ago

I'm missing the entity select field. Another person in chat found it on drupal. https://chat.civicrm.org/civicrm/pl/sw55rjmqktn5tyxsjpbutea63c Civi 5.76.1, csv_import 1.13, wordpress 6.6.1 image

ConfigAndLog Error

2024-08-20 11:47:12-0500  [warning] entity parameter expected, always passed in core & few outside core uses so this will go Caller: CRM_Csvimport_Import_Controller::__construct
Array
(
    [civi.tag] => deprecated
)

PHP Error

[Tue Aug 20 11:47:14.973254 2024] [proxy_fcgi:error] AH01071: Got error 'PHP message: PHP Warning:  Undefined variable $contactType in /bitnami/wordpress/wp-content/plugins/civicrm/civicrm/CRM/Contact/Page/AJAX.php on line 406', ....
Levi-70 commented 2 months ago

This happened with a recent update. the issue is on Drupal as well. I tested on dmaster demo site 5.78.alpha1 it also happens on my site which is currently on 5.76.1 it was ok when using 5.74.4. The first time I noticed it is after upgrading from 5.74.4 to 5.76.1 so somewhere btw those civi versions

savionlee commented 2 months ago

There's something happening by the time getHookedTemplateFileName() in Core QuickForms that is picking up the normal import template instead of the extension's template on newer civi.

here's some rough screenshots, see line 114 for the different templateFile working template call, civi 5.74.5 working_template-call-2

not working template call, civi 5.76.1 broken_template-call-2

savionlee commented 2 months ago

There's something happening by the time getHookedTemplateFileName() in Core QuickForms that is picking up the normal import template instead of the extension's template on newer civi.

here's some rough screenshots, see line 114 for the different templateFile

I need to check how that $page attribute is being passed in. The not working shows its calling the core import instead of csvimport a couple lines before the render template