Problem/Motivation

Looks like ModuleHandler.php depends on "Doctrine\Common\Inflector\Inflector" (doctrine/inflector). I could not find this class in Drupal Core. I think we should add a composer.json

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 2598570-4.patch376 bytesbenjy

Comments

webflo created an issue. See original summary.

webflo’s picture

webflo commented
Issue summary: View changes
benjy’s picture

benjy commented

Adding a composer.json sounds fine.

Inflector is in Doctrine and Drupal has a dependency on that? Did you run into an error here somehow?

benjy’s picture

benjy commented
Status: Active » Needs review
StatusFileSize
new 2598570-4.patch376 bytes

Maybe we can use this issue to introduce the composer.json file, how does the attached look?

benjy’s picture

benjy commented

Just noticed a trailing comma, will remove that on commit.

benjy’s picture

benjy commented

Adding webflo to credits.

benjy’s picture

benjy commented
Title: Missing dependency to doctrine/inflector » Add a composer.json file
benjy’s picture

benjy commented
Status: Needs review » Fixed

Committed.

  • benjy committed 244655f on 8.x-1.x 
    Issue #2598570 by benjy, webflo: Add a composer.json file

Status: Fixed » Needs work

The last submitted patch, 4: 2598570-4.patch, failed testing.

benjy’s picture

benjy commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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