return 'flagtype_entity';

We removed the 'flagtype' prefix a while back: #2481793: EntityFlagTypeDeriver $ignoredEntities won't allow contrib modules to define their own flag types.

CommentFileSizeAuthor
#4 2581423-4.flag_.flagtype-entity-test-helper-method.patch371 bytesjoachim
#2 2581423.flag_.flagtype-entity-test-helper-method.patch316 bytesjoachim

Comments

joachim created an issue. See original summary.

joachim’s picture

joachim commented
Status: Active » Needs review
StatusFileSize
new 2581423.flag_.flagtype-entity-test-helper-method.patch316 bytes
socketwench’s picture

socketwench commented
Status: Needs review » Needs work

I'd like a comment explaining the default return, otherwise it looks good.

joachim’s picture

joachim commented
Status: Needs work » Needs review
StatusFileSize
new 2581423-4.flag_.flagtype-entity-test-helper-method.patch371 bytes

Done.

socketwench’s picture

socketwench commented
Status: Needs review » Reviewed & tested by the community

Works for me!

joachim’s picture

joachim commented
Status: Reviewed & tested by the community » Fixed

Thanks!

  • joachim committed 3d597a3 on 8.x-4.x 
    Issue #2581423 by joachim: Fixed getFlagType() returning an invalid flag...

Status: Fixed » Closed (fixed)

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