Working on it.
This issue depends on another issue
link
🐛
drupal_get_path() is deprecated in Drupal 9.3.0 and removed from Drupal 10.0.0
RTBC
so we need to apply changes of this issue first to use this theme otherwise it will throw error.
So by apply the changes of the above issue and this MR
Blockquote css have been applied correctly and the changes are present in both the files (.scss and .css).
So moving it to RTBC.
Arun,
Thanks
The MR works fine and the spinner is appearing as expected so moving it to RTBC.
Working on it.
Added test,
@smustgrave please review.
Hi @arpitr,
I've updated the help text, please review.
Working on it.
Added a logger, please check.
Working on it.
Hi @murz
The paragraph is center alligned now so moving it to RTBC.
Working on it.
above MR fixed the issue, Working for me . moving RTBC++
Hi @finex,
Could you please provide the step to reproduce the issue as I've enabled both the modules(Layout Builder, LinkChecker) and I've addded a link field in the Article and managed its layout through layout builder but I didn't get any error.
Hi @alex9000,
This warning is due to the dynamic property $moduleHandler being assigned directly in the constructor of LinkCheckerAdminSettingsForm.
So we require a docblock comment in order to remove the warning.
I see that that docblock is present in 2.0.x and 2.1.x but it is not present when you do composer require 'drupal/linkchecker:^2.0'
so you can use these versions instead or you can explicitly write this docblock.
/**
* ModuleHandler service.
*
* @var \Drupal\Core\Extension\ModuleHandlerInterface
*/
protected $moduleHandler;
@vladimiraus please check this.
Thanks,
Arun
Working on it.
Created an MR and also fixed the pipeline.
arunsahijpal → made their first commit to this issue’s fork.
Please check now.
@joachim,
Could you please review it.
@tstoeckler,
Now what should we do?
Created a MR please check.
arunsahijpal → made their first commit to this issue’s fork.
Hi @kiwad,
I've installed the experience builder on drupal cms project but I didn't get any error.
Hi @dcoppel,
I followed the steps you mentioned but didn't got any error, could you please elaborate the steps you mentioned.
Thanks,
Arun
I've updated the HtmlConfigurationFormForm.php with backward compatibility. pls check.
I've updated all the images mentioned above except the logo because its already small in size (13kb), also I've updated the configuration of respective images, but the pipeline is failing due to these changes so could you pls check the changes and pipeline.
Opened MR and added the file, pls check.
Hi @greg-boggs,
I've compressed the image and converted it into webp and now it is around 82kb.
The drupal file uploader doesn't allow me share it in the comment section as it doesn't support .webp format.
So how can I share the image could you pls tell.
@tstoeckler,
Could please suggest me some changes to fix the pipeline.
The MR changes are working fine.
Attaching ss for reference.
Working on it.
Hi @laurentb,
I've updated the checkAccess method and now even when you set "View own profiles" without giving the edit access to the user you can see the fields for the profile, and for the field permission module could please create a separate issue, I've a solution for that too.
Thanks,
Arun
Hi @berdir,
are you talking about this line" class="!text-blue-400 !no-underline" target="_blank" rel="nofollow" >line → ?
Working on it.
@godotislate,
I ran the pipeline again buy it is still failing.
Ok @greg-boggs, I understood.
It would be very helpful if could you pls tell me the approach we can follow here, as I was thinking to update createDerivative method in Image module but we can't do this here as it is Drupal cms not core.
Hi @godotislate,
I've updated the docblock, pls check.
Working on it.
Hi @greg-boggs,
I've tried to reproduce this by following the steps you have mentioned, but I didn't get the images of large sizes.
I think, as this
issue
🐛
An image in latest content loads an extremely large image
Active
is fixed so the problem is solved now.
If something is left please do let me know.
Thanks,
Arun
@peterwcm,
I've updated the code and now no more orphaned paragraphs and everything saves when the node saves by referencing the entity directly instead of manually setting target_id.
and I also think this condition !$item->entity->isNew()
is not necessary.
For the translation thing there is one limitation that - Paragraphs are referenced entities, not simple text fields.
Drupal does not natively support translating entity reference fields.
When I tried to translate it shows this error
* (unsupported) Paragraphs fields do not support translation
. Yo can checkout this link → for more info .
I think there is one tot check translation with Paragraphs Library module but I'm not sure.
Could you pls check it, if it works then I think we can safely remove that condition.
Working on it.
Hi @peterwcm,
I've updated the createDuplicate method please check.
Also @berdir pls check is this a good solution?
Thanks,
Arun
Hi @phenaproxima,
That makes sense—mixing test-related logic into production code with drupal_valid_test_ua()
can make the codebase harder to maintain.
So I'm planning to Introduce a service to determine test mode (TestModeChecker
), and will replace drupal_valid_test_ua()
calls with service usage.
Would it be a good solution?
Hi @liam-morland,
please check now.
Hi @marianrk,
I've added a hook_entity_update to delete the referenced paragraphs please check.
Also @berdir please check whether this should be the part of module behaviour or not.
Thanks,
Arun
Working on it.
@liam-morland,
I've changed the targeted branch.
Working on it.
Hi @nginex,
I've added a backward compatibility of old approach, pls check.
Hi @liam-morland,
I have reproduced the issue on my local and got the same warning,but after applying the patch the warning disappears.
So making a MR for it.
pls check.
Thanks
Arun.
Working on it.
@hansfn,
Could pls look into this.
Hi @kunalratnani,
I've created a taxonomy named Category and referenced it in 2 CT (Article and Basic Page), and
I've applied your MR(patch) but is only showing "None" value for the referencing field and I can't see separate counts for Article and Basic page, so moving it to NW.
Attaching ss for reference.
@aleksandarborosgyevi pls look into this whether its a useful feature or not.
Thanks,
Arun
Changed the status by mistake to RTBC
moving back to NW.
@smustgrave ,
pls check now
Good work @sandip-poddar,
I've reproduced the issue on my local and by applying the MR the multilevel menu problem is fixed.
Attaching ss for reference.
Thanks,
Arun
Good work @rajat4u ,
I tested this issue on my local and by applying the MR the issue is completely resolved as maintenance page is shown without clearing the cache.
Attaching ss for reference.
Hence moving it to RTBC.
Thanks
Arun
Hi @lazzyvn,
I've followed the same steps but i didn't get any error.
@bojanz could pls look into this
Thanks
I've installed it using composer
I've also tried to reproduced this issue but nothing happened in my case .
Attaching ss for reference.
Hi @chrisolof,
I've removed the hook_search_api_query_alter() and added a event subscriber instead pls check.
Thanks
Working on it.
Hi @jaydeep_patel,
Could you pls check now.
@bohart could you please suggest some method.
Working on it.
@narendrar, pls check
Working on it.
Hi @murz,
Your changes are good and working fine but we also need to do the same thing in importFieldValue
function in order to import content smoothly, so I've made the necessary changes, pls check.
Thanks,
Arun
Working on it.
Hi @perryjanssen,
This solution is working fine for me as ConfirmFormInterface do not require entity locking since they do not modify entity data directly.
Attaching ss for reference.
But we would like to here best solution from @alexpott
Thanks
arunsahijpal → made their first commit to this issue’s fork.
@chsdaiguil I think there may be some issue at your end or maybe you are using multilingual site that's why it is happening, I am not sure.
@bramdriesen and @berdir please look into this.
I meant- Check screenshot(ss) for more information.
@joachim
I've made the changes in the original MR and hide the new MR, please check.
arunsahijpal → changed the visibility of the branch 3497021-deprecate-getFormName to hidden.
arunsahijpal → changed the visibility of the branch 3497021-confirmforminterfacegetformname-serves-no to active.
Hi @chsdaiguil,
It doesn't show 1 vote when its 0
see ss for reference
Hi @cebab54,
I've sent mail using this module and I didn't see any warning so could you pls tell me the steps to reproduce this and how can I add tokens to this?
@joachim,
I've made the changes that you suggested, pls review.
Hi @liam-morland,
I've tried to add dependency using calculateDependencies()
method present in BehaviorSettings.php
file but it doesn't adding the dependency although I've tried it by modifying schema but it didn't work too , so it would be a great help if you could suggest me some approach that how can I add dependency to it.
Thanks
Working on it.
arunsahijpal → changed the visibility of the branch 3497021-confirmforminterfacegetformname-serves-no to hidden.
@joachim
Could you please provide me the Change Record link so that I can use it.
@smustgrave,
It would be great help if you could suggest me some change so that the pipeline could pass as it is bit problematic for me.
Hi @quietone and @joachim,
So should i revert my changes and add a deprecation comment instead?
@smustgrave,
pls review now.
Hi @tstoeckler and @smustgrave ,
I've added the test with batch and also I've updated the content
key check, pls review
Nice Work @gaurav-gupta,
As mentioned on clicking the checkbox the effects are not being applied to its derivative image styles but after applying the efffects are also applied on derivative image styles.
Sharing the recording for reference.
Hence moving it to RTBC.
Thanks,
Arun
The changes LGTM
But @niharika.s I was already assigned to this issue and was working on it, so please check before you start working on an issue.
Working on it.
@tstoeckler,
I've updated the code as you have asked, plc check.