🇪🇸Spain @eduardo morales alberti

Spain, 🇪🇺
Account created on 26 September 2017, over 7 years ago
#

Merge Requests

More

Recent comments

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Could you provide steps to reproduce it?
We have logged the same error, but we were unable to reproduce it.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

RTBC!! We tested it using the debug configuration, and now it only processes the saved translation once (not the default language). Take into account that the process entity is called from the node save, so it will take as reference the language of the saved node, not the default translation.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

The current maintainer is not working with Drupal anymore, so it is possible that we will not get any response.

We uninstalled the module and replaced the configuration.

Steps to it:

  • Create a new content display with the same configuration as content_browser display, we called it entity_browser.
  • On views, replace content browser field, by a render field using the new entity_browser display.
  • Replace any custom styling or twig from content_browser to entity_browser, example node--content-browser.html.twig => node--entity-browser.html.twig
  • Once it is replaced, uninstall the module and check if any configuration needs to be recovered (it should not, as we removed all dependencies of content_browser from views and node display)

When the module is uninstalled it will remove the views with the content_browser field, and the entity_browsers with those views, so it is important to replace the field before uninstall it, also it will remove the content_browser display, that's why we created a new one with the same config.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

@stefan.korn on Issue 🐛 Function file_validate_image_resolution is deprecated in drupal:10.2.0 and is removed from drupal:11.0.0. Active you have a more complete solution compatible with Drupal 10 and 11.

We created a new module to remove the EXIF information, but just because some images save geolocation about where it was taken and device information, but is optional and not related to exif_orientation, it is just to give an option to strip the information without losing the rotation.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

There are more vulnerable packages https://www.drupal.org/project/gin/issues/3529722 🐛 Update Vulnerable npm Packages Active

+ ddev exec COLUMNS=200 cd web/themes/contrib/gin; npm audit

[31mFailed to execute command `COLUMNS=200 cd web/themes/contrib/gin; npm audit`: exit status 1[0m
# npm audit report

@babel/helpers  <7.26.10
Severity: moderate
Babel has inefficient RegExp complexity in generated code with .replace when transpiling named capturing groups - https://github.com/advisories/GHSA-968p-4wvh-cqc8
fix available via `npm audit fix`
node_modules/@babel/helpers

@babel/runtime  <7.26.10
Severity: moderate
Babel has inefficient RegExp complexity in generated code with .replace when transpiling named capturing groups - https://github.com/advisories/GHSA-968p-4wvh-cqc8
fix available via `npm audit fix`
node_modules/@babel/runtime

brace-expansion  2.0.1 - 4.0.0
brace-expansion Regular Expression Denial of Service vulnerability - https://github.com/advisories/GHSA-v6h2-p8h4-qcjw
fix available via `npm audit fix`
node_modules/svg-spritemap-webpack-plugin/node_modules/brace-expansion
  minimatch  5.0.0 - 9.0.5 || >=10.0.1
  Depends on vulnerable versions of brace-expansion
  node_modules/svg-spritemap-webpack-plugin/node_modules/minimatch
    glob  8.0.1 - 10.4.5
    Depends on vulnerable versions of minimatch
    node_modules/svg-spritemap-webpack-plugin/node_modules/glob
      svg-spritemap-webpack-plugin  >=4.4.1
      Depends on vulnerable versions of glob
      node_modules/svg-spritemap-webpack-plugin

cross-spawn  <6.0.6 || >=7.0.0 <7.0.5
Severity: high
Regular Expression Denial of Service (ReDoS) in cross-spawn - https://github.com/advisories/GHSA-3xgq-45jj-v275
Regular Expression Denial of Service (ReDoS) in cross-spawn - https://github.com/advisories/GHSA-3xgq-45jj-v275
fix available via `npm audit fix`
node_modules/cross-spawn
node_modules/webpack-cli/node_modules/cross-spawn

elliptic  <=6.6.0
Severity: critical
Valid ECDSA signatures erroneously rejected in Elliptic - https://github.com/advisories/GHSA-fc9h-whq2-v747
Elliptic's private key extraction in ECDSA upon signing a malformed input (e.g. a string) - https://github.com/advisories/GHSA-vjh7-7g9h-fjfh
fix available via `npm audit fix`
node_modules/elliptic

nanoid  <3.3.8
Severity: moderate
Predictable results in nanoid generation when given non-integer values - https://github.com/advisories/GHSA-mwcw-c2x4-8c55
fix available via `npm audit fix`
node_modules/nanoid

postcss  <=8.4.30
Severity: moderate
Regular Expression Denial of Service in postcss - https://github.com/advisories/GHSA-566m-qj78-rww5
PostCSS line return parsing error - https://github.com/advisories/GHSA-7fh5-64p2-3v2j
No fix available
node_modules/postcss-perfectionist/node_modules/postcss
node_modules/postcss-scss/node_modules/postcss
  postcss-perfectionist  *
  Depends on vulnerable versions of postcss
  Depends on vulnerable versions of postcss-scss
  node_modules/postcss-perfectionist
  postcss-scss  <=1.0.6
  Depends on vulnerable versions of postcss
  node_modules/postcss-scss

12 vulnerabilities (4 low, 6 moderate, 1 high, 1 critical)

To address issues that do not require attention, run:
  npm audit fix
🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Duplicated of https://www.drupal.org/project/gin/issues/3529723 🐛 Update Vulnerable npm Packages Active

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Hi!
Any news on this issue?
We had the same problem on our Drupal with duplicated users

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

@sdhruvi5142 We are not sure if the test failed because the reCAPTCHA is for testing purposes on your video.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

It is possible that is an error related to the static analysis from https://www.drupal.org/project/paragraphs/issues/3492718 🐛 Call to deprecated method loadRevision() of class Drupal\Core\Entity\EntityStorageInterface errors Active

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Removed Drupal 9 support and added Drupal 11 to the .info

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Added context, and once on recatcha validation.

@mahde could you confirm that it solves your issue?

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

We sent him an email, let's wait

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Site structure

Menus: /en/admin/reports/xray-audit/site-structure/menu

We changed the view to show all menus on the same table to make it easy to download.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

@lpeidro
We changed to tabs Entities Displays, as they were different operations, and to maintain the standard of one operation and a download button per page.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Example of report CSV download:

Added download button on almost all reports, except XrayAuditQueryTaskImageStylesPlugin as the current data structure was too difficult to adapt.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

ZipArchiver does not works properly on Drupal 9.5, needs patch from issue https://www.drupal.org/project/drupal/issues/2850794 🐛 Unable to open Zip archive using ArchiverZip Fixed

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

We had the same problem with CloudFlare as it sorts the parameters and leads on the oembed items not loading.
The change solves it.
We added a hook update to clear the caches, because the hook post update only is launched if there is any update pending to be executed.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Remove internal links as already have a section on targets.
Move external links to targets.
Add schemas like mail, tel.. to targets => Other.
Add sections on targets:
Entities
Views
Others/Schema
...

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Added new configuration to choose which content will be processed

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

@himanshu_jhaloya The patch does not works because the validation is done before the method removeUtilityClassValue is called, so the form state already have required issues.

Also, cover removeUtility button.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

The problems is solved by clicking on "Check manually", so the projects that have been uninstalled can be ignored.

https://www.drupal.org/project/drupal/issues/3270052#comment-14628951 🐛 Undefined array key in prepareUpdateData Active

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Maybe the patch should also removed the uninstalled projects if were removed.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Same problem here https://www.drupal.org/project/drupal/issues/3270052#comment-16089374 🐛 Undefined array key in prepareUpdateData Active

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

In our case seems like the "locale.translation_status" has modules that were removed from the project but not from the key:

 
$status = \Drupal::keyValue('locale.translation_status')->getAll();
 
$module_handler = \Drupal::service('module_handler');
$module_handler->loadInclude('locale', 'inc', 'locale.compare');
$modules = \Drupal::service('extension.list.module')->getList();
$themes = \Drupal::service('extension.list.theme')->getList();
$project_list = array_merge($modules, $themes);
print_r(array_diff(array_keys($status), array_keys($project_list)));

Those modules were already removed from Drupal:

Array
(
    [0] => acquia_connector
    [1] => acquia_search
    [3] => addtoany
    [5] => advagg
    [9] => blazy
    [12] => colorbox
    [13] => colorbox_media_video
    [15] => config_filter
    [18] => config_update
    [19] => consumers
    [27] => email_registration
    [56] => jquery_ui_datepicker
    [58] => jsonapi_earlyrendering_workaround
    [59] => jsonapi_extras
    [64] => maillog
    [69] => memcache
    [74] => migrate_plus
    [107] => simple_oauth
    [115] => transliterate_filenames
)

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

We made some changes, including adding an OperationTimeCompressingHandler, to compress the logs, as they can be heavy on large sites.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

We maintained the following list as the following attributes were not on the module:

spacing
spacing-y
spacing-x
headings
font-size
font-family
  • text-align
    • left
    • right
    • center
  • color
    • primary
    • secondary
  • background-color
    • primary
    • secondary
  • btn
    • primary
    • secondary
    • success
    • info
    • warning
    • danger
  • btn-size
    • large
    • small
🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

On the custom theme option we will keep only the following utilities as a base:

buttons
badge
text-align
text-color
bg-color
spacing
spacing-y
spacing-x
headings
font-size
font-family
🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Currently, the log entities are saved on a file archived_entity-YY-MM-DD.csv, but the date is when the file was created, and then, it does not match with the entity_logs, change the rotating file handler to use the entity logs dates.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Duplicated of https://www.drupal.org/project/log_entity_operations/issues/3520644 Add the hability to recover removed entity log operations Active

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

@luke.leber We are not sure how to contribute a test for it, as we are removing the attribute id, so we can not test that the id is duplicated, as it does not exist.

Do you refer to creating a test of the current code, testing the ID duplication?

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Solved PHPStan and PHPCS warnings, pending CSPell and stylelint

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

PHPStan Errors:

 ------ --------------------------------------------------------------------------- 
  Line   modules/vlsuite_block/modules/vlsuite_block_cta/vlsuite_block_cta.install  
 ------ --------------------------------------------------------------------------- 
  38     Access to constant BUNDLE_KEY on an unknown class VLSuiteBlockCta.         
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols        
  38     Class VLSuiteBlockCta not found.                                           
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols        
 ------ --------------------------------------------------------------------------- 
 ------ ----------------------------------------------------------------------------- 
  Line   modules/vlsuite_block/modules/vlsuite_block_icon/vlsuite_block_icon.install  
 ------ ----------------------------------------------------------------------------- 
  38     Access to constant BUNDLE_KEY on an unknown class VLSuiteBlockIcon.          
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols          
  38     Class VLSuiteBlockIcon not found.                                            
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols          
  45     Access to constant BUNDLE_KEY on an unknown class VLSuiteBlockIcon.          
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols          
  45     Class VLSuiteBlockIcon not found.                                            
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols          
 ------ ----------------------------------------------------------------------------- 
 ------ ------------------------------------------------------------------------------- 
  Line   modules/vlsuite_block/modules/vlsuite_block_image/vlsuite_block_image.install  
 ------ ------------------------------------------------------------------------------- 
  27     Access to constant BUNDLE_KEY on an unknown class VLSuiteBlockImage.           
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols            
  27     Class VLSuiteBlockImage not found.                                             
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols            
 ------ ------------------------------------------------------------------------------- 
 ------ ---------------------------------------------------------------------------- 
  Line   modules/vlsuite_block/src/Controller/VLSuiteBlockChooseBlockController.php  
 ------ ---------------------------------------------------------------------------- 
  20     Class                                                                       
         Drupal\vlsuite_block\Controller\VLSuiteBlockChooseBlockController           
         extends @internal class                                                     
         Drupal\layout_builder_restrictions\Controller\ChooseBlockController.        
 ------ ---------------------------------------------------------------------------- 
 ------ ------------------------------------------------------------------- 
  Line   modules/vlsuite_block/src/Plugin/Block/VLSuiteFieldBlock.php       
 ------ ------------------------------------------------------------------- 
  29     Class Drupal\vlsuite_block\Plugin\Block\VLSuiteFieldBlock extends  
         @internal class Drupal\layout_builder\Plugin\Block\FieldBlock.     
 ------ ------------------------------------------------------------------- 
 ------ -------------------------------------------------------------------- 
  Line   modules/vlsuite_block/src/Plugin/Block/VLSuiteInlineBlock.php       
 ------ -------------------------------------------------------------------- 
  28     Class Drupal\vlsuite_block\Plugin\Block\VLSuiteInlineBlock extends  
         @internal class Drupal\layout_builder\Plugin\Block\InlineBlock.     
 ------ -------------------------------------------------------------------- 
 ------ -------------------------------------------------------------------------- 
  Line   modules/vlsuite_block/src/Plugin/Derivative/VLSuiteFieldBlockDeriver.php  
 ------ -------------------------------------------------------------------------- 
  13     Class Drupal\vlsuite_block\Plugin\Derivative\VLSuiteFieldBlockDeriver     
         extends @internal class                                                   
         Drupal\layout_builder\Plugin\Derivative\FieldBlockDeriver.                
 ------ -------------------------------------------------------------------------- 
 ------ --------------------------------------------------------------------------------- 
  Line   modules/vlsuite_block/src/Plugin/Derivative/VLSuiteMediaBgFieldBlockDeriver.php  
 ------ --------------------------------------------------------------------------------- 
  14     Class                                                                            
         Drupal\vlsuite_block\Plugin\Derivative\VLSuiteMediaBgFieldBlockDeriver           
         extends @internal class                                                          
         Drupal\layout_builder\Plugin\Derivative\FieldBlockDeriver.                       
  27     \Drupal calls should be avoided in classes, use dependency injection             
         instead                                                                          
 ------ --------------------------------------------------------------------------------- 
 ------ ---------------------------------------------------------------- 
  Line   modules/vlsuite_block/vlsuite_block.module                      
 ------ ---------------------------------------------------------------- 
  185    Fetching deprecated class constant EXISTS_REPLACE of interface  
         Drupal\Core\File\FileSystemInterface:                           
         in drupal:10.3.0 and is removed from drupal:12.0.0. Use         
         \Drupal\Core\File\FileExists::Replace instead.                  
 ------ ---------------------------------------------------------------- 
 ------ --------------------------------------------------------------------- 
  Line   modules/vlsuite_collection/src/VLSuiteCollectionHelper.php           
 ------ --------------------------------------------------------------------- 
  96     Call to static method encode() on an unknown class                   
         Drupal\Core\Serialization\Yaml.                                      
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols  
  120    Call to static method decode() on an unknown class                   
         Drupal\Core\Serialization\Yaml.                                      
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols  
  193    Fetching deprecated class constant EXISTS_REPLACE of interface       
         Drupal\Core\File\FileSystemInterface:                                
         in drupal:10.3.0 and is removed from drupal:12.0.0. Use              
         \Drupal\Core\File\FileExists::Replace instead.                       
 ------ --------------------------------------------------------------------- 
 ------ --------------------------------------------------------------------- 
  Line   modules/vlsuite_demo/vlsuite_demo.install                            
 ------ --------------------------------------------------------------------- 
  127    Call to static method decode() on an unknown class                   
         Drupal\Core\Serialization\Yaml.                                      
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols  
 ------ --------------------------------------------------------------------- 
 ------ ------------------------------------------------------------------------------------------ 
  Line   modules/vlsuite_generator/src/Drush/Generators/VLSuiteGeneratorModuleGenerator.php        
 ------ ------------------------------------------------------------------------------------------ 
  136    Parameter $default_content_exporter of method                                             
         Drupal\vlsuite_generator\Drush\Generators\VLSuiteGeneratorModuleGenerator::__construct()  
         has invalid type Drupal\default_content\ExporterInterface.                                
 ------ ------------------------------------------------------------------------------------------ 
 ------ ------------------------------------------------------------------------------------------------ 
  Line   modules/vlsuite_layout/modules/vlsuite_layout_tabs/src/Plugin/Layout/VLSuiteLayoutTabsBase.php  
 ------ ------------------------------------------------------------------------------------------------ 
  167    \Drupal calls should be avoided in classes, use dependency injection                            
         instead                                                                                         
 ------ ------------------------------------------------------------------------------------------------ 
 ------ ----------------------------------------------------------------- 
  Line   modules/vlsuite_modal/src/Form/VLSuiteModalSettingsForm.php      
 ------ ----------------------------------------------------------------- 
  129    Call to deprecated method clearCachedDefinitions() of interface  
         Drupal\Core\Asset\LibraryDiscoveryInterface:                     
         in drupal:11.1.0 and is removed from drupal:12.0.0. Use          
         LibraryDiscoveryCollector::clear() instead.                      
 ------ ----------------------------------------------------------------- 
 ------ ---------------------------------------------------------------------- 
  Line   modules/vlsuite_modal/src/VLSuiteModalConfigOverride.php              
 ------ ---------------------------------------------------------------------- 
  101    \Drupal calls should be avoided in classes, use dependency injection  
         instead                                                               
 ------ ---------------------------------------------------------------------- 
 [ERROR] Found 23 errors                                                    
🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

CSpell

| Original        | Corrected         |
|----------------|-------------------|
| acive          | active            |
| activado       | activated         |
| animatecss     | Animate.css       |
| autocollapse   | auto-collapse     |
| autoresize     | auto-resize       |
| behaviour      | behavior          |
| botom          | bottom            |
| builderlink    | builder link      |
| cardinallity   | cardinality       |
| clasees        | classes           |
| clases         | classes           |
| clientside     | client-side       |
| completly      | completely        |
| contentvlsuite | content VLSuite   |
| cpath          | cPath             |
| crect          | correct           |
| csvg           | CSVG              |
| dependecies    | dependencies      |
| depenencies    | dependencies      |
| deprecatedfield| deprecated field  |
| desactivado    | deactivated       |
| descarted      | discarded         |
| dialogclose    | dialog close      |
| dinamically    | dynamically       |
| dynamize       | dynamize (consider rephrasing) |
| everytning     | everything        |
| fourcols       | four columns      |
| galleryvlsuite | gallery VLSuite   |
| groupelement   | group element     |
| indentifier    | identifier        |
| initiazilation | initialization    |
| laning         | planning          |
| limititations  | limitations       |
| linuxfoundation| Linux Foundation  |
| nagination     | pagination        |
| navlink        | nav link          |
| olumn          | column            |
| opctions       | options           |
| otherwhise     | otherwise         |
| overing        | hovering          |
| palcehold      | placeholder       |
| placehold      | placeholder       |
| preffix        | prefix            |
| proccesed      | processed         |
| procesing      | processing        |
| provinding     | providing         |
| quoute         | quote             |
| reduntant      | redundant         |
| remoto         | remote            |
| resovler       | resolver          |
| scrollpy       | ScrollSpy         |
| scrope         | scope             |
| selecto        | selector          |
| spicific       | specific          |
| swiperjs       | Swiper.js         |
| sysyem         | system            |
| tada           | Tada (animation)  |
| textdirection  | text direction    |
| threecols      | three columns     |
| twocols        | two columns       |
| udate          | update            |
| uidom          | UI DOM            |
| usement        | usage             |
| utilit         | utility           |
| utilitites     | utilities         |
| utilitties     | utilities         |
| utilty         | utility           |
| utitlities     | utilities         |
| vluiste        | VLSuite           |
| vídeo          | video             |
| warantee       | warranty          |
| webforms       | Web Forms         |
| wher           | where             |
| whitout        | without           |

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Ready to review:
On a fresh installation:

  • Just access to /admin/config/vlsuite/utility-classes
  • Configure the framework
  • If it selects Bootstrap or Tailwind, it will load default utility classes
  • If it selects Custom, it will load the same utility classes as Bootstrap but empty, so each utility class will need to be filled or removed.
  • If the site already has configuration, the form will set bootstrap5 as the default framework
🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Ready to review:
On a fresh installation:

  • Just access to /admin/config/vlsuite/utility-classes
  • Configure the framework
  • If it selects Bootstrap or Tailwind, it will load default utility classes
  • If it selects Custom, it will load the same utility classes as Bootstrap but empty, so each utility class will need to be filled or removed.
  • If the site already has configuration, the form will set bootstrap5 as the default framework
🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

All errors were fixed on the main branch using tools like PHPCBF and Stylelint.

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

Thank you, @omarlopesino, for reconsidering the feature

🇪🇸Spain eduardo morales alberti Spain, 🇪🇺

The problem comes from \Drupal\entity_mesh\Repository::getLabel
As it gets the label from the entity, but not from the translation

  protected function getLabel(?string $entity_type, ?string $entity_id): ?string {
    if (empty($entity_id) || empty($entity_type)) {
      return NULL;
    }

    try {
      $storage = $this->entityTypeManager->getStorage($entity_type);
    }
    catch (PluginNotFoundException $e) {
      return NULL;
    }

    /** @var \Drupal\Core\Entity\EntityInterface $entity */
    $entity = $storage->load($entity_id);
    if (!$entity instanceof EntityInterface) {
      return NULL;
    }

    return $entity->label();
  }
Production build 0.71.5 2024