Moving back to active, as dependent issue was fixed.
Access is revoked now. That's good.
Thank you. That helps. Will send an initial MR soon.
I need some guidance on it. One lodash function we use is debounce. Are we supposed to create a new custom debounce function?
Should we close this issue since 2.x is released?
Adding related issue so that the link is present.
I can't reproduce it now either.
I would like to take this up.
@joachim Should I create another issue for the same?
Removed unwanted code. Tests are passing.
Test failure was a random error. Now working fine.
Merge request is updated to 11.x branch. It has a test failure though. Needs further analysis.
anish.a → changed the visibility of the branch 3246454-add-weekly-granularity to hidden.
anish.a → changed the visibility of the branch 3246454-add-weekly-granularity to active.
anish.a → changed the visibility of the branch 3246454-views-weekly-granularity to hidden.
anish.a → changed the visibility of the branch 3246454-views-weekly-granularity to active.
anish.a → changed the visibility of the branch 3246454-add-weekly-granularity to hidden.
anish.a → changed the visibility of the branch 3246454-add-weekly-granularity to active.
I was trying to rebase with 11.x. Some functions are deprecated. Not sure which way should I go to fix this.
anish.a → changed the visibility of the branch 3246454-add-weekly-granularity to hidden.
Adding screenshot of working module from D11.
anish.a → changed the visibility of the branch 3434141-automated-drupal-11 to hidden.
anish.a → changed the visibility of the branch project-update-bot-only to hidden.
Drupal 11 compatibility is already in 1.x-dev and this issue is outdated. We just need a release.
There is a call to deprecated function getEntityTranslation
. This needs to be fixed.
anish.a → changed the visibility of the branch 3495766-drupal-11-compatibility to hidden.
Can we do a release for Drupal 11 compatibility?
Adding the fix. Empty string instead of NULL fixes the warning.
I am using 2.0.0-beta5.
I am not sure whether it happened after the beta. I think I selected some wrong options and got locked out of the page.
These are the warnings I was seeing in the Drupal messages.
Warning: Undefined array key "#row" in Drupal\entityreference_filter\Plugin\views\filter\EntityReferenceFilterViewResult->processOptions() (line 211 of modules/contrib/entityreference_filter/src/Plugin/views/filter/EntityReferenceFilterViewResult.php).
Error message
Warning: Attempt to read property "_entity" on null in Drupal\entityreference_filter\Plugin\views\filter\EntityReferenceFilterViewResult->processOptions() (line 211 of modules/contrib/entityreference_filter/src/Plugin/views/filter/EntityReferenceFilterViewResult.php).
I tried debugging and found that this is getting as $renderable
in processOptions
. This obviously is not the array it is expecting.
Array
(
[0] => views_view__entity_reference_view__entityreference_4
[1] => views_view__entityreference_4
[2] => views_view__default
[3] => views_view__entity_reference_view__default
[4] => views_view__default
[5] => views_view__entity_reference_view
[6] => views_view
)
I am stuck here. Sending the configuration file for the view.
From what I read, the code can be removed. Let me reproduce the issue on Drupal 9.0.x.
Adding the reference issue.
You just removed that comment!
What was the context for the comment? What change was needed?
I had submitted a rough draft and I need some help on finishing it.
Changing the status and assigning to me as I am continuing working on this.
Created a draft MR. I still need to fix the dependency injection errors, continuing the work.
I am taking this up. I will push the changes on what I had done by today itself.
@rahul_ Can you push the WIP in the branch, so that I can take a look.
Working on this
@lavanyatalwar It is already being worked on by @rahul_ in DrupalCon Singapore and he is already in finishing stages. I suggest to take up another issue to avoid duplicate work.
Tests are missing and we need to write tests to verify the functionality.
This will work only after the related issue is fixed.
I am taking it up.
FYI, these are the errors
30/30 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
------ ------------------------------------------------------------------------------
Line quicktabs_accordion/src/Plugin/TabRenderer/AccordionTabs.php
------ ------------------------------------------------------------------------------
54 \Drupal calls should be avoided in classes, use dependency injection instead
------ ------------------------------------------------------------------------------
------ ------------------------------------------------------------------------------
Line quicktabs_jqueryui/src/Plugin/TabRenderer/UiTabs.php
------ ------------------------------------------------------------------------------
27 \Drupal calls should be avoided in classes, use dependency injection instead
------ ------------------------------------------------------------------------------
------ -----------------------------------------------------------------------------------
Line src/Controller/QuickTabsController.php
------ -----------------------------------------------------------------------------------
48 Unsafe usage of new static().
💡 See: https://phpstan.org/blog/solving-phpstan-error-unsafe-usage-of-new-static
------ -----------------------------------------------------------------------------------
------ -----------------------------------------------------------------------------------
Line src/Form/QuickTabsInstanceEditForm.php
------ -----------------------------------------------------------------------------------
49 Unsafe usage of new static().
💡 See: https://phpstan.org/blog/solving-phpstan-error-unsafe-usage-of-new-static
------ -----------------------------------------------------------------------------------
------ -----------------------------------------------------------------------------------
Line src/Plugin/Block/QuickTabsBlock.php
------ -----------------------------------------------------------------------------------
50 Unsafe usage of new static().
💡 See: https://phpstan.org/blog/solving-phpstan-error-unsafe-usage-of-new-static
------ -----------------------------------------------------------------------------------
------ ------------------------------------------------------------------------------
Line src/Plugin/Derivative/QuickTabsBlock.php
------ ------------------------------------------------------------------------------
18 \Drupal calls should be avoided in classes, use dependency injection instead
------ ------------------------------------------------------------------------------
------ ------------------------------------------------------------------------------
Line src/Plugin/TabRenderer/QuickTabs.php
------ ------------------------------------------------------------------------------
74 \Drupal calls should be avoided in classes, use dependency injection instead
------ ------------------------------------------------------------------------------
------ ------------------------------------------------------------------------------
Line src/Plugin/TabType/BlockContent.php
------ ------------------------------------------------------------------------------
67 \Drupal calls should be avoided in classes, use dependency injection instead
69 \Drupal calls should be avoided in classes, use dependency injection instead
75 \Drupal calls should be avoided in classes, use dependency injection instead
81 \Drupal calls should be avoided in classes, use dependency injection instead
97 \Drupal calls should be avoided in classes, use dependency injection instead
98 \Drupal calls should be avoided in classes, use dependency injection instead
121 \Drupal calls should be avoided in classes, use dependency injection instead
122 \Drupal calls should be avoided in classes, use dependency injection instead
------ ------------------------------------------------------------------------------
------ --------------------------------------------------------------------------------------------------------------------
Line src/Plugin/TabType/NodeContent.php
------ --------------------------------------------------------------------------------------------------------------------
35 \Drupal calls should be avoided in classes, use dependency injection instead
60 \Drupal calls should be avoided in classes, use dependency injection instead
62 Method Drupal\quicktabs\Plugin\TabType\NodeContent::render() should return string but return statement is missing.
64 \Drupal calls should be avoided in classes, use dependency injection instead
70 \Drupal calls should be avoided in classes, use dependency injection instead
------ --------------------------------------------------------------------------------------------------------------------
------ ------------------------------------------------------------------------------
Line src/Plugin/TabType/QtabsContent.php
------ ------------------------------------------------------------------------------
27 \Drupal calls should be avoided in classes, use dependency injection instead
48 \Drupal calls should be avoided in classes, use dependency injection instead
------ ------------------------------------------------------------------------------
------ ------------------------------------------------------------------------------
Line src/Plugin/TabType/ViewContent.php
------ ------------------------------------------------------------------------------
76 \Drupal calls should be avoided in classes, use dependency injection instead
81 \Drupal calls should be avoided in classes, use dependency injection instead
86 \Drupal calls should be avoided in classes, use dependency injection instead
89 \Drupal calls should be avoided in classes, use dependency injection instead
102 \Drupal calls should be avoided in classes, use dependency injection instead
180 \Drupal calls should be avoided in classes, use dependency injection instead
------ ------------------------------------------------------------------------------
------ --------------------------------------------------------------------------------------------------------------------
Line src/Plugin/views/style/Quicktabs.php
------ --------------------------------------------------------------------------------------------------------------------
5 Interface Drupal\Core\Form\FormStateInterface referenced with incorrect case: Drupal\core\form\FormStateInterface.
------ --------------------------------------------------------------------------------------------------------------------
[ERROR] Found 29 errors
Removing Needs Screenshots
I think this fixes the issue mentioned and we can raise a feature ticket for Adding custom field for title. Let me create a ticket and link to this.
Drupal 11 version is already released.
Tested the issue and is ready for merge.
anish.a → changed the visibility of the branch 3241489-fix-fullcalendarfieldisdate-re to hidden.
Updating the branch as its still present in latest DEV
anish.a → made their first commit to this issue’s fork.
For MacOS
$ cd /path/to/drupal/root $ drush cex -y $ cd /path/to/config $for f in $(ls core*title.yml); do sed -i '' 's,^required: false,required: true,' $f; done
$ drush cim -y
I represented Drupal Thiruvananthapuram Community as a panelist.
I am able to reproduce this issue. Will try to reproduce with only Core, Registration and Conditional Fields
alexpott → credited anish.a → .
No longer requires reroll.
Removing needs-reroll as it is already rerolled.
There are test failures. I am unsure about how to fix it.
Adding for review again after rebase.
We are taking up this issue in DrupalCampPune2024
Just changing core version won't work
Duplicate of #3433705
The issue raised by heikkiy is already addressed by issue #3269746. This works fine and can be merged.
This can be merged. Tested and working fine
Any pointers to fix the issue where -Any- appears last?
Adding a custom migration yml file that I used to test this.
Adding more details.
Please see the attached screenshot. Eventhough the D7 site has fields populated, The source comes out as empty. I checked using debugging turned on.
Not sure how to proceed further.
I am facing the same issue. How did you fix that?
This patch is not applying on latest pathauto.
I was having same issue with the webform_validation. I had to manually downgrade views_templates module also.
It would be good if you can merge and make it compatible to D10.
#71 no longer applies
These patches no longer applies
I apologize for the same. I had to use drush generate
to generate the module and forms. This was developed for the usage in multiple websites for a client and we decided to publish it so that others can benefit.
I understand that the module is too small to opt into security coverage. So, I am closing the task.
Sorry, I don't have any other modules/themes listed in drupal.org. Is there any approximate number of lines in the code to qualify for a review?