Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Using Composer for installation is more and more becoming the standard. Yet, the libraries are still be uploaded the manual way using i.e. filezilla
Proposed resolution
create a composer.libraries.json file so the libraries can be uploaded using the wikimedia composer-merge-plugin
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#9 | 3185456-6.patch | 1.28 KB | bart lambert |
#5 | 3185456-5.patch | 1.54 KB | Pooja Ganjage |
#4 | Schermafbeelding 2020-12-02 190449.png | 13.46 KB | bart lambert |
#2 | 3185456-2.patch | 1.53 KB | Pooja Ganjage |
|
Issue fork flexslider-3185456
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:
Comments
Comment #2
Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commentedHi,
Creating a patch for this issue.
Please review the patch.
Let me know if any suggestions.
Thanks.
Comment #3
Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commentedComment #4
bart lambert CreditAttribution: bart lambert commentedtested but got an error that download failed:
Comment #5
Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commentedUpdated patch.
Please review.
Thanks.
Comment #6
bart lambert CreditAttribution: bart lambert commentedtested and working on php 7.4 & MySQL 5.7 D8.9.10
Comment #7
bart lambert CreditAttribution: bart lambert commentedComposer is actually being update to version 2.0 which poses problems with the wikimedia/composer-merge-plugin:^1.4.1.
Persons using composer 2.0 have to use:
wikimedia/composer-merge-plugin:"dev-feature/composer-v2 as 1.5.0"
So in the composer.libraries.json file line 20:
"wikimedia/composer-merge-plugin": "^1.4.1" should be replaced by
"wikimedia/composer-merge-plugin": "dev-feature/composer-v2 as 1.5.0"
I don't know if this newer version is backwards compatible with the composer version 1
Comment #8
bart lambert CreditAttribution: bart lambert commentedANother Hickup!
Using composer-merge-plugin and the proposed composer.libraries.json file, the library is installed in the libraries-folder in the folder woocommerce.flexslider and actually it needs the foldername flexslider
Comment #9
bart lambert CreditAttribution: bart lambert commentedComment #12
Ruslan PiskarovSeams as @bart lambert added a json file instead of a patch. I have committed the fix.
Comment #13
Ruslan PiskarovComment #14
bart lambert CreditAttribution: bart lambert commentedthanks!
Comment #15
liquidcms CreditAttribution: liquidcms commentedComposer can't be used to apply this patch, correct? Chicken/egg issue. So the only was to use this is if it gets committed?
Comment #16
Ruslan Piskarov@liquidcms, you can use with composer https://git.drupalcode.org/project/flexslider/-/merge_requests/1.diff.
Comment #17
euk CreditAttribution: euk commentedWould be great to have this added to the D10 tagged version...
Comment #20
sleitner CreditAttribution: sleitner commentedPlease review MR11 for 3.x
Comment #22
rossb89 CreditAttribution: rossb89 at ComputerMinds commentedI updated MR-11 with the latest changes in 3.x as it was a bit out of date and not mergeable.
I'm not quite sure why we have both MR-1 *and* MR-11 though?
Update - oh sure it's because MR-11 was done for 3.x but it's the same exact branch as MR-1 was for 2.x 😅
Comment #23
Ruslan PiskarovDeleted comment. Sory.