When using asset-packagist for downloading clipboard,js library, the package is downloaded in `libraries/clipboard` folder instead of `libraries/clipboard.js`.
Here is a patch to change the library path.

CommentFileSizeAuthor
change-library-path.patch410 bytessharique

Issue fork clipboardjs-3293959

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Sharique created an issue. See original summary.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Status: Needs review » Needs work

We might be able to leverage hook_library_info_alter to support the old and the new path for backwards compatibility:
https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Render%21...

david.muffley’s picture

david.muffley commented

Similarly when installing the library through an explicit composer package it goes into /libraries/clipboardjs (no dot).

Edited incorrect composerjs to correct clipboardjs.

tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented

I'm much more a fan of using the asset-packagist then using the merge plugin. So if someone has time and interest to fix this, would be great.

tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented
Version: 2.0.x-dev » 2.1.x-dev

Lets support this in 2.1.x

tim-diels opened merge request !22

tim-diels opened merge request !23

tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented
Status: Needs work » Needs review

  • tim-diels committed 6032e317 on 2.1.x 
    Issue #3293959 by tim-diels, sharique, sanduhrs: When using asset-...

  • tim-diels committed 33dcabba on 2.0.x 
    Issue #3293959 by tim-diels, sharique, sanduhrs: When using asset-...
tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.