Opened 6 years ago

Closed 6 years ago

#68 closed defect (fixed)

Move the plugin code to its own class

Reported by: tsee Owned by:
Priority: major Milestone:
Component: editor Version:
Keywords: Cc:

Description

In the wake of adding the plugin-from-PAR functionality, it struck me that the plugin code (which can only grow) should really be in its own class.

Hence, I propose the Padre::PluginManager? class (not Padre::Plugin::Manager for a reason).

The attached patch implements the move and does some cleanup at the same time. Unfortunately, this relies on ticket #66 (PAR support) and #67 (autocompletion) since I made the change to my modified checkout. Sorry.

The attached patch includes those of #66 and #67 as well. I don't think it's separable from #66, but I'll happily produce a patch without the changes in #67 on demand.

This change should make it easier to move the plugin-loading to the time *after* the window has been opened. It's only a single method call. (see ticket #1)

Attachments (1)

ticket_66_67_68.diff (10.5 KB) - added by tsee 6 years ago.
Patch implementing this ticket, as well as 66/67

Download all attachments as: .zip

Change History (2)

Changed 6 years ago by tsee

Patch implementing this ticket, as well as 66/67

comment:1 Changed 6 years ago by tsee

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.