🇸🇬Singapore @anish.a

The website encountered an unexpected error. Try again later.

ArgumentCountError: Too few arguments to function Drupal\hello_login\HelloClient::__construct(), 6 passed in /var/www/html/web/core/lib/Drupal/Component/DependencyInjection/Container.php on line 259 and exactly 7 expected in Drupal\hello_login\HelloClient->__construct() (line 106 of /var/www/html/src/HelloClient.php).

Drupal\Component\DependencyInjection\Container->createService() (Line: 177)
Drupal\Component\DependencyInjection\Container->get() (Line: 430)
Drupal\Component\DependencyInjection\Container->resolveServicesAndParameters() (Line: 237)
Drupal\Component\DependencyInjection\Container->createService() (Line: 177)
Drupal\Component\DependencyInjection\Container->get() (Line: 430)
Drupal\Component\DependencyInjection\Container->resolveServicesAndParameters() (Line: 237)
Drupal\Component\DependencyInjection\Container->createService() (Line: 177)
Drupal\Component\DependencyInjection\Container->get() (Line: 430)
Drupal\Component\DependencyInjection\Container->resolveServicesAndParameters() (Line: 249)
Drupal\Component\DependencyInjection\Container->createService() (Line: 177)
Drupal\Component\DependencyInjection\Container->get() (Line: 36)
Drupal\hello_login\Controller\HelloLoginController::create() (Line: 36)
Drupal\Core\DependencyInjection\ClassResolver->getInstanceFromDefinition() (Line: 100)
Drupal\Core\Utility\CallableResolver->getCallableFromDefinition() (Line: 34)
Drupal\Core\Controller\ControllerResolver->getControllerFromDefinition() (Line: 49)
Drupal\Core\Controller\ControllerResolver->getController() (Line: 166)
Symfony\Component\HttpKernel\HttpKernel->handleRaw() (Line: 76)
Symfony\Component\HttpKernel\HttpKernel->handle() (Line: 53)
Drupal\Core\StackMiddleware\Session->handle() (Line: 48)
Drupal\Core\StackMiddleware\KernelPreHandle->handle() (Line: 28)
Drupal\Core\StackMiddleware\ContentLength->handle() (Line: 32)
Drupal\big_pipe\StackMiddleware\ContentLength->handle() (Line: 201)
Drupal\page_cache\StackMiddleware\PageCache->fetch() (Line: 138)
Drupal\page_cache\StackMiddleware\PageCache->lookup() (Line: 87)
Drupal\page_cache\StackMiddleware\PageCache->handle() (Line: 48)
Drupal\Core\StackMiddleware\ReverseProxyMiddleware->handle() (Line: 51)
Drupal\Core\StackMiddleware\NegotiationMiddleware->handle() (Line: 53)
Drupal\Core\StackMiddleware\AjaxPageState->handle() (Line: 51)
Drupal\Core\StackMiddleware\StackedHttpKernel->handle() (Line: 715)
Drupal\Core\DrupalKernel->handle() (Line: 19)

Stack tracke

Test failure is there. I can fix it.

Co-ordinated directly with droptimes. Sebin from droptimes can vouch.

Co-ordinated directly with droptimes. Sebin from droptimes can vouch.

Co-ordinated directly with droptimes. Sebin from droptimes can vouch.

Session room -2.33, two sessions

An early exit could be cleaner.

anish.a → created an issue.

anish.a → created an issue.

anish.a → created an issue.

anish.a → created an issue.

anish.a → created an issue.

Pipeline still fails. Not sure if it is correct.

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?

anish.a → created an issue.

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.

anish.a → made their first commit to this issue’s fork.

@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 11.x 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 → made their first commit to this issue’s fork.

anish.a → changed the visibility of the branch 3246454-add-weekly-granularity to hidden.

anish.a → made their first commit to this issue’s fork.

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.

anish.a → created an issue.

Can we do a release for Drupal 11 compatibility?

Adding the fix. Empty string instead of NULL fixes the warning.

anish.a → created an issue.

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.

anish.a → created an issue.

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.

anish.a → created an issue.

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                                                                                                
                                                                                                                        

anish.a → made their first commit to this issue’s fork.

Removing Needs Screenshots

anish.a → created an issue.

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.

anish.a → created an issue.

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.

anish.a → made their first commit to this issue’s fork.

We are taking up this issue in DrupalCampPune2024

Just changing core version won't work