When updating to 2.0.2, the following error is thrown:

[Drupal\Core\Extension\Exception\UnknownExtensionTypeException]  
  Extension type library is unknown.

This is due to line 50 in file_browser_requirements of file_browser.install

  if (\Drupal::moduleHandler()->moduleExists('libraries')) {
    $imagesloaded_path = \Drupal::service('extension.path.resolver')->getPath('library', 'imagesloaded') . '/imagesloaded.pkgd.min.js';
    $masonry_path = \Drupal::service('extension.path.resolver')->getPath('library', 'masonry') . '/dist/masonry.pkgd.min.js';
  }

Specifically, the getPath('library' call

Issue fork file_browser-3544437

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

daniel.moberly created an issue. See original summary.

rcodina made their first commit to this issue’s fork.

rcodina opened merge request !14

rcodina’s picture

rcodina
Primary language Catalan
Location Barcelona
commented
Status: Active » Needs review

Just created MR to fix the issue. Remember to install library dependencies using Wikimedia Composer Merge Plugin.

d.fisher’s picture

d.fisher commented
Version: 2.0.2 » 2.x-dev
Status: Needs review » Reviewed & tested by the community

Thank you for this. Looks good to me. Marking RTBC.

caspervoogt’s picture

caspervoogt commented

that solved it for me

rcodina’s picture

rcodina
Primary language Catalan
Location Barcelona
commented
Status: Reviewed & tested by the community » Needs review

I've done some changes to avoid duplicated code.

d.fisher’s picture

d.fisher commented

This looks good to me. @caspervoogt can you test the latest changes in MR!14 as well and if we both see it solving the problem I'm happy to get this one merged and released.

d.fisher’s picture

d.fisher commented
Status: Needs review » Reviewed & tested by the community

Marking RTBC based on my testing and no response.

d.fisher’s picture

d.fisher commented
Status: Reviewed & tested by the community » Fixed

Merged to 2.x and will be in next tagged release.

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

dhansen’s picture

dhansen commented
Status: Fixed » Needs work

Reopening. The changes fixed this issue in the install file, but not in the .module file where the libraries are actually called for use.

dhansen opened merge request !17

dhansen’s picture

dhansen commented
Status: Needs work » Needs review

The MR is working for me, but would like someone else with Libraries API installed to give it a try.

d.fisher’s picture

d.fisher commented
Status: Needs review » Reviewed & tested by the community
d.fisher’s picture

d.fisher commented

Tested and looks good. Merging now.

d.fisher’s picture

d.fisher commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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