Closed (fixed)
Project:
Display fields
Version:
9.1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Jun 2022 at 09:30 UTC
Updated:
18 Jul 2022 at 12:29 UTC
Jump to comment: Most recent
Comments
Comment #2
delta commentedhi,
can you check the updated branch or last release?
thank you
Comment #3
pski commentedHi,
Thanks for your work! This error is solved, but now, the following appears during
drush updb:ParseError: syntax error, unexpected ')', expecting ']' in Composer\Autoload\includeFile() (line 306 of /var/www/web/modules/contrib/display_fields/src/Element/EntityFieldSelect.php) #0 phar:///usr/local/bin/drush/.box/vendor/composer/ClassLoader.php(322): Composer\Autoload\includeFile('/var/www/web/mo...')It has been fixed on 8.x branch but not on 9.x.
Comment #4
avpadernoComment #5
avpadernoLet's fix this on the latest branch, first.
I am checking the namespaces used from the classes and the directories where their files have been placed. I think I found a discrepancy.
Comment #6
avpadernoAll the classes/interfaces are placed in the right directory and they are using the correct namespace.
The src/Element/EntityFieldSelect.php file doesn't contain any syntax error. They have been fixed.
Comment #7
delta commentedpublished https://www.drupal.org/project/display_fields/releases/9.0.0-beta4
thanks Alberto!
Comment #8
pski commentedHi,
Thanks for these changes.
However, I still have an error in dblog after updating to 9.0.0-beta4:
Error : Class 'Drupal\display_fields\Plugin\DefaultFactory' not found in Drupal\display_fields\Plugin\DisplayFieldsPluginManager->createInstance() (/var/www/web/modules/contrib/display_fields/src/Plugin/DisplayFieldsPluginManager.php ligne 37)You should add
use Drupal\Component\Plugin\Factory\DefaultFactory;insrc/Plugin/DisplayFieldsPluginManager.php.Comment #11
immaculatexavier commentedAddressed #8 in issue fork #10, Please review
Comment #12
avpadernoComment #13
avpadernoComment #14
avpadernoComment #17
avpadernoThis has been fixed in both the branches. Thank you for the report and the issue fork!
Comment #19
delta commentedthank you, published in https://www.drupal.org/project/display_fields/releases/9.0.0-beta5