Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warnings in drupal container startup #44

Open
emetsger opened this issue Jan 8, 2021 · 0 comments
Open

Warnings in drupal container startup #44

emetsger opened this issue Jan 8, 2021 · 0 comments
Labels
wontfix This will not be worked on

Comments

@emetsger
Copy link

emetsger commented Jan 8, 2021

There are a number of warnings that are accumulating in the Drupal container startup log, and I'm just creating this issue to track their origin and potential resolution.

Masonry

drupal_1      |  [error]  Content Browser requires the Masonry library. Download the newest release from
drupal_1      | https://github.com/desandro/masonry/releases and place it in /libraries

This error is tied to the missing Masonry JS library and other missing JS artifacts like the PDF viewer that never made it into idc-isle-dc from the original drupal site. This error will be resolved when Masonry is added.

This error has been present since idc-isle-dc was inaugurated.

SimpleTest

drupal_1      |  [warning]  (Currently using Testing (SimpleTest) The <em>Testing</em> (SimpleTest) module is deprecated for removal in Drupal 9. It should not be enabled on production sites. Read <a href="https://www.drupal.org/node/3091784">The Drupal core SimpleTest module is deprecated</a> for alternative ways to run tests during development.)

This error seems to be related to the presence of a deprecated module. It isn't clear whether Drupal or Islandora depend on this module, or if it can be removed without consequence.

This error, too, has been around since idc-isle-dc was inaugurated.

EasyRdf\Graph

drupal_1      |  [warning] Cannot declare class  \EasyRdf\Graph, because the name is already in use IFedoraApi.php:24

This error is newer, and I think was introduced in 517f87b, though I am not 100% positive.

The error seems benign, but kind of smells like EasyRdf is being depended upon by two different modules. If that is the case, it may be worth noting in case the version of EasyRdf diverges (at worst, incompatible versions of the library are depended upon).

@htpvu htpvu added the wontfix This will not be worked on label Nov 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants