- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 6:36am 10 February 2023 - 🇮🇳India spriya2896
I have fixed the issues mentioned in #12. Please review
- Status changed to Needs work
almost 2 years ago 3:20pm 10 February 2023 - 🇮🇳India harshitthakore Pune
Noticed existing code got change with new methods and approach in base branch. So as of now sync with base branch and we can move forward.
- Status changed to Needs review
over 1 year ago 9:41am 19 October 2023 - Status changed to Needs work
7 months ago 12:01pm 15 July 2024 - 🇮🇳India riddhi.addweb
Please review the Patch it is failing, When I applied the Patch it failed in the 9.5x version.
Checking patch addtoany.libraries.yml... Checking patch addtoany.module... error: while searching for: * Handle AddToAny integration. */ use Drupal\Core\Entity\EntityInterface; use Drupal\Core\Entity\Display\EntityViewDisplayInterface; use Drupal\Core\Routing\RouteMatchInterface; use Drupal\Core\Render\Markup; use Drupal\Core\Url; use Drupal\Component\Utility\UrlHelper; use Drupal\node\Entity\Node; use Drupal\addtoany\Form\AddToAnySettingsForm; /** * Implements hook_help(). error: patch failed: addtoany.module:5 error: addtoany.module: patch does not apply Checking patch js/addtoany.admin.js... error: while searching for: (function ($) { 'use strict'; Drupal.behaviors.addtoany = { attach: function (context, settings) { // Initial page setup var addtoany_icon = $('input[name="addtoany_universal_button"]').next('label').find('img:first'); var initial_icon_size_int = parseInt($('input[name="addtoany_buttons_size"]').val()); // Set the A2A icon's size to match selected Icon Size addtoany_icon.height(initial_icon_size_int).width(initial_icon_size_int); // Bring attention to large A2A icon option // because the universal button will likely be changed to match the other icons // (Drupal #states can't handle this) $('input[name="addtoany_buttons_size"]').change(function () { var icon_size = $(this).val(); var icon_size_int = parseInt(icon_size); // Set the A2A icon's size to match selected Icon Size addtoany_icon.height(icon_size_int).width(icon_size_int); }); } }; }(jQuery)); error: patch failed: js/addtoany.admin.js:1 error: js/addtoany.admin.js: patch does not apply Checking patch js/addtoany.js... Checking patch src/Form/AddToAnySettingsForm.php... error: while searching for: * @param \Drupal\Core\Extension\ExtensionList $module_extension_list * The module extension list service. * @param \Drupal\Core\Entity\EntityTypeBundleInfoInterface $entity_type_bundle_info * The entity type bundle info service. */ public function __construct(ConfigFactoryInterface $config_factory, ModuleHandlerInterface $module_handler, ExtensionList $module_extension_list, EntityTypeBundleInfoInterface $entity_type_bundle_info) { parent::__construct($config_factory); error: patch failed: src/Form/AddToAnySettingsForm.php:48 error: src/Form/AddToAnySettingsForm.php: patch does not apply Checking patch src/Plugin/Block/AddToAnyBlock.php... error: while searching for: use Drupal\Core\Block\BlockBase; use Drupal\Core\Form\FormStateInterface; use Drupal\node\NodeInterface; /** * Provides an 'AddToAny share' block. error: patch failed: src/Plugin/Block/AddToAnyBlock.php:4 error: src/Plugin/Block/AddToAnyBlock.php: patch does not apply Checking patch src/Plugin/Block/AddToAnyFollowBlock.php... error: src/Plugin/Block/AddToAnyFollowBlock.php: No such file or directory Checking patch src/Plugin/views/field/NodeAddToAnyShare.php...
- First commit to issue fork.
- 🇺🇸United States DamienMcKenna NH, USA
I can confirm that MR 3329621-fix-coding-standards-2.x solves some coding errors I was setting in 2.0.11, thank you.