Use Composer Manager for easy and clean install [#2828789]

The documentation for install is not clear at all, and also there is a lot of duplicate files due to composer (vendor/sebastian, /vendor/symfony, etc..) into the libraries folder.

Why not implement Brightcove module with the Composer Manager module so we will be able to:
1/ Install it with Drush in one line
2/ Avoid install problems and troubleshoot
3/ Avoid hundreds of duplicate files
4/ Manage easily the update for security issues by Composer Manager

https://www.drupal.org/project/composer_manager

Comment	File	Size	Author
#12	brightcove-composer_manager-2828789-12.patch	329 bytes	sergiu.savva
#7	error.PNG	8.85 KB	Vj

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

18 November 2016 at 12:48

kobee created an issue. See original summary.

Comment #2

Vj CreditAttribution: Vj as a volunteer commented 17 April 2017 at 19:16

Yes agreed documentation is not clear at all. Installed brightcove module on clean install.

Steps followed till now

Installed brightcove module
Unzip the PHP MAPI Wrapper package into this directory and rename it to "brightcove".
https://github.com/brightcove/PHP-API-Wrapper/ added to libraries and run composer install
When visited admin/config/media/brightcove/playlist/add gives error "Fatal error: Class 'Brightcove\Object\Playlist' not found in sites/all/modules/contrib/brightcove/brightcove.playlist.inc on line 24"

As suggested kobee can we include composer manger here to make install clean & something like hook_requirements which will take care of all packages are installed correctly before module enabled.

Thanks,
Vj

Comment #3

tamasd CreditAttribution: tamasd at Pronovix commented 19 May 2017 at 11:22

I am open to this, but I don't want to drop the libraries support. When I will have some budget, I will look around how this can be implemented.

Comment #4

Vj CreditAttribution: Vj as a volunteer commented 20 May 2017 at 15:29

Hey tamasd,

Can you please help me to resolve error "Fatal error: Class 'Brightcove\Object\Playlist' not found in sites/all/modules/contrib/brightcove/brightcove.playlist.inc on line 24".

Comment #5

Anonymous (not verified) CreditAttribution: Anonymous commented 22 May 2017 at 21:13

NOT TO HIJACK THE ISSUE

We are having nothing but trouble installing this module on a clean 7.54 Drupal install with all of the dependencies and Bootstrap theme I keep getting Brightcove client is not found. It will not write the API information.

Comment #6

jan.mashat CreditAttribution: jan.mashat at Pronovix for BrightCove commented 23 May 2017 at 04:55

Vj, which documentation is not clear?
And where exactly did you find 7.x-6.x install steps telling you to do the following?

Unzip the PHP MAPI Wrapper package into this directory and rename it to "brightcove".

weavingmedia, have you tried searching for that error? Here is one result for example.

In case anyone hasn't found the installation guide, it's linked under Documentation in the sidebar of the project page.

Comment #7

Vj CreditAttribution: Vj as a volunteer commented 23 May 2017 at 07:35

File	Size
error.PNG	8.85 KB

Hey jan.mashat,

Followed steps 1 to 3 from https://www.drupal.org/docs/7/modules/brightcove-video-connect-7x-6x/ins... on clean drupal site.

On module enable It throws error "Fatal error: Cannot use variable $client twice in \sites\all\modules\contrib\brightcove\brightcove.video.inc on line 107".

Module versions:
brightcove-7.x-6.4.zip
ctools-7.x-1.12.zip
date-7.x-2.10.zip
entity-7.x-1.8.zip
libraries-7.x-2.3.zip

Library version:
PHP-API-Wrapper-1.0.zip

Thanks,
Vj

Comment #8

tamasd CreditAttribution: tamasd at Pronovix commented 23 May 2017 at 07:48

Vj: that is a known issue, and it is fixed in dev. It is a simple typo, if you need a quick fix just apply this patch: http://cgit.drupalcode.org/brightcove/patch/?id=4501a064460108abe0e6f653...