Let me give i a try
Hi,
I have created a patch which displays a warning, when $form['#empty_value'] exist in the form and is set to 1 as I saw that the $form['#empty_option'] replaces the $form['#option'] only when $form['#empty_value'] is set to 1. Please review the patch.
Hi,
I have created a logo and added in the issue.
Hi,
We can add '#description' property as condition if we have '#empty_option' property in the form. This will conditionally render the description below the form field. Refer the example given below:-
/**
* {@inheritdoc}
*/
public function buildForm(array $form, FormStateInterface $form_state): array {
$form['select_element'] = [
'#type' => 'select',
'#options' => [
0 => 'zero',
1 => 'one',
2 => 'two'
],
'#required' => TRUE,
'#empty_value' => TRUE,
'#empty_option' => "Some empty value"
];
if($form['select_element']['#empty_option']) {
$form['select_element']['#description'] = 'Empty option present will replace the option.';
}
$form['actions'] = [
'#type' => 'actions',
'submit' => [
'#type' => 'submit',
'#value' => $this->t('Send'),
],
];
return $form;
}
Hi @ultrabob,
Thank you for your suggestion but if we use ',' instead of '+' in permission filter then, to enable 'Structure' link in the toolbar, we will require both the permissions 'access administration pages,access taxonomy overview'. And the user role having both the permission will only be able to access the 'Structure' link. But 'access administration pages' permission should only be given to 'Administrator' role rather than other roles. So I think , its better that we use '+' instead of ',' in the permission filter.
Please let me know your thoughts.
Hi,
I have worked in the issue and created a patch. I have tested the patch in 10.2.x and added the following permission to allow the user access the taxonomy pages:-
1. Tags: Edit terms (or whatever vocabulary)
2. Access the taxonomy vocabulary overview page.
Hi,
I have worked in the issue mentioned above and created the patch. Please review the patch.
Hi,
I have created a patch for the issue mentioned where I have changed the descriptions in system_token_info(). Could you please review it and provide the feedback.
Hi,
I have created a patch which will add 'manage patterns' permission in permission page. The role having the permission will give an user the ability to create/edit/delete/enable/disable/duplicate pathauto patterns and use the bulk generation form, but will withdraw the ability to edit the settings configuration. Please review and let me know.
Hi,
I have made few changes in the patch submitted above in #2. Could you please review and provide your feedback.
Hi,
I have changed the token description and re generated the patch. I have tested the patch in Drupal 10.0.12-dev and the patch was working properly in my system. Could you please review the patch
Hi,
I have changed the media type description and added in patch. Please review the changes.
Hi,
I have created the logo again. Can you please review it.
Hi,
I created a logo for Advanced CSS/JS Aggregation (advagg). Please review it.
Hi,
I created a patch as per comment #9, and I am getting the same timeout result after 5 incorrect attempts. Are we expecting this result or there is anything else to be done. Please review it and let me know.
anweshasinha → made their first commit to this issue’s fork.
Hi,
I have made the changes in my patch according to #111 and have regenerated the patch again. Please do review it once and let me know.
Hi,
I have created patch for the search api autocomplete issue received above. Please review once.
Hi,
Added patch for the issue according to php 8.1.22 standard. Please review the patch
Hi,
I created a logo for drupal commerce which includes Centarro logo. Please review it.
Hi,
I have made some more changes in the code based on #108 and have regenerated the patch again. Please do review it once and let me know.
Hi,
I have made necessary changes by adding apace beside the dropdown and created a patch in drupal 9.5.x. I have attached the patch and the interdiff file. Please review the patch and let me know.
Hi,
I have contributed a module in drupal.org in drupal 9 https://www.drupal.org/project/document_to_pdf → . You can check it.
Hi,
I have made some more changes in the code based on #103 and have regenerated the patch again. Please do review it once and let me know.
Hi,
I have applied the patch from #94(project_link_9.5.x_dev_94.patch) in drupal 11.x version as the core version has changed to 11.x. So I made the necessary changes regarding the project link and attached the interdiff and the patch file. Please review it.
I am working on this