- Issue created by @mydot
- 🇮🇳India kishan@lnwebworks
Hi @mydot
I installed the module in Drupal version 10.4.3 and PHP version 8.3.13.
Linked field Module is working fine in the local environment.Can you please share the steps to replicate the issue?
- 🇮🇳India kishan@lnwebworks
Hello @mydot.
I reviewed the module on my local environment.
Config page is working fine without any errorsCould you please share the steps to replicate the issue.
I have attached my drupal version and PHP version.
As mentioned i use Drupal 11.1.3. and not Drupal 10 as you. Maybe there is the problem
- 🇮🇳India kishan@lnwebworks
@mydot Thanks for mentioning the Drupal version.
I replicated the issue in Drupal 11 version and provided the merge request ,and it's working fine.
Please review once. - First commit to issue fork.
- 🇦🇺Australia tarawij
Hi @kishan kumar and maintainers,
Thank you for the great work on MR #17! I have tested the current changes and noticed a few issues related to constructor arguments and service injection for D11 support.
To help move this forward, I made some adjustments and fixes, including:
- Corrected the constructor signature and service injection to align with D11 standards.
- Improved the order of dependencies in the factory method.
- Simplified the row calculation logic for better readability.
- Updated the row calculation comment for clarity.
You can find my commit with the fixes here:
Fix commit for MR #17Since I couldn't push changes to the main project repository to directly update the MR, I kindly ask if you could cherry-pick my commit or incorporate the changes into the existing MR. Please let me know if you would like me to make any further adjustments.
Thank you once again for your effort on this! I hope my contribution helps move the MR forward.
- First commit to issue fork.
Can confirm error in D11 and that the config form loads with additional changes from @tarawij