We have moved to Github. Please open tickets there.

Opened 2 years ago

Closed 21 months ago

#3434 closed Defect (fixed)

Invalid plugin manifests break the plugin admin area

Reported by: brettp Owned by:
Priority: normal Milestone: Elgg 1.8.0
Component: Core Version: 1.8 Beta
Severity: minor Keywords:
Cc: brett@… Difficulty: easy

Description

Causes an invalid plugin warning.

Change History (3)

comment:1 Changed 2 years ago by cash

Add a '&' to an enabled plugin and crash the site:

Fatal error: Call to a member function getConflicts() on a non-object in /opt/trunk/engine/classes/ElggPluginPackage.php on line 337

comment:2 Changed 21 months ago by brettp

  • Summary changed from Using & in a plugin manifest breaks the plugin in advanced section to Invalid plugin manifests break the plugin admin area

comment:3 Changed 21 months ago by Brett Profitt

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

Fixes #3434. Manifests are checked more carefully on anything that checks deps. Disabling plugins with invalid manifests from admin page.

Changeset: 0179e8c68b0827d77c61a31c8c0d6bf4a277c785

Note: See TracTickets for help on using tickets.