Usually example modules is optional, but in this module, the config link is only provided by 'Overlay paths - Example', this is not resonable.

Comments

dddbbb’s picture

Yeah, this does seem a bit odd. I actually couldn't figure out how I was supposed to use this module until I found this issue (thanks @rogical).

A 'Configure' link next to the module row when it's enabled in admin/modules would also be a great help, especially for new users.

steven jones’s picture

Sorry for the confusion, this module isn't really designed to have a UI, it should be used by module developers, the example module is just that: an example.

Agreed that this could do with some further clarification.

steven jones’s picture

Status: Active » Fixed

Clarified the project page text.

aasarava’s picture

Steven, have you considered renaming overlay_paths_example to overlay_paths_ui, so that it's an optional UI module?

steven jones’s picture

I have, but overlay paths is an API module, not a UI module. If someone wants to start such a project, then go for it!

Chris Charlton’s picture

Would you accept the patch that splits the example module into both a `overlay_paths_ui` module, and an example module?

dddbbb’s picture

+1 for the above patch idea. While this module was not intended to ship with a UI, surely there's no harm in including one if the community are willing to deliver it? Especially if there's a demand (which there appears to be).

steven jones’s picture

Title: Can't use this module if not enable module 'Overlay paths - Example' » Create a UI
Version: 7.x-1.0 » 7.x-1.x-dev
Category: bug » feature
Status: Fixed » Active

Okay okay, so clearly you guys want a UI module, fair enough.

We can basically rename the example module, and re-purpose it I guess.

dddbbb’s picture

Wahaaay! :D

steven jones’s picture

Status: Active » Fixed

Pushed to 7.x-1.x.

Status: Fixed » Closed (fixed)

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

Chris Charlton’s picture

Sweet. I would recommend updating the project's description to reflect the change. If there is no example module to be shipped (now or in the future), then referencing the UI also as an example module would make sense (or adding a *.api.php file).

Thanks again.

steven jones’s picture

Thanks @Chris Charlton I've made those changes.