We have moved to Github. Please open tickets there.

Opened 2 years ago

Closed 20 months ago

#3229 closed Enhancement (fixed)

Elgg needs a better error message for plugins failing to activate

Reported by: danelund Owned by:
Priority: normal Milestone: Elgg 1.8.1
Component: Core Version: Github Master
Severity: minor Keywords:
Cc: brett@… Difficulty: easy

Description

Stating what is the reason the plugin is failin.

Change History (8)

comment:1 Changed 2 years ago by cash

  • Difficulty set to easy
  • Milestone changed from Needs Review to Elgg 1.8

Assigning this to the 1.8 milestone. Thanks for the report.

comment:2 Changed 2 years ago by cash

(In [svn:8903]) Refs #3229 at least we can forward so that user sees error message immediately

comment:3 Changed 2 years ago by cash

Ugh - activating plugins has the same issue that logging in has - many ways to fail but a function that just returns false. The way we dealt with that for login is to throw an exception when an error occurs. That way we can pass a message back to the action to display. Thoughts?

comment:4 Changed 2 years ago by brettp

The ElggPlugin objects do that in other places. Makes sense to do it for here too...

comment:5 Changed 2 years ago by ewinslow

  • Milestone changed from Elgg 1.8 to Elgg 1.8.1

Feels like more enhancement than defect. Pushing this back.

comment:6 Changed 2 years ago by ewinslow

  • Type changed from Defect to Enhancement

comment:7 Changed 2 years ago by brettp

Note for the future: This is mostly just to expose the errors that are already thrown.

comment:8 Changed 20 months ago by brettp

  • Resolution set to fixed
  • Status changed from new to closed

This was fixed in [58b66643c1921706d764b2bb1ea4729519ca31f8]

Note: See TracTickets for help on using tickets.