#3804 closed Defect (fixed)
Standardize "External Pages" vs "Site Pages" references
| Reported by: | bchapin | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | Elgg 1.8.1b |
| Component: | Core | Version: | 1.8 |
| Severity: | trivial | Keywords: | |
| Cc: | brett@… | Difficulty: | easy |
Description
External pages are enabled via the External Pages plugin. The pages are configured in the Admin/Appearance/Site Pages menu. Seems confusing. Probably should be named consistently
Change History (4)
comment:1 Changed 21 months ago by brettp
- Difficulty set to easy
- Milestone changed from Needs Review to Elgg 1.8.1b
- Priority changed from normal to low
- Severity changed from minor to trivial
- Summary changed from External Pages plugin name/ Admin menu Appearance/Site Pages confusing to Standardize "External Pages" vs "Site Pages" references
comment:2 Changed 21 months ago by cash
We have two plugins: externalpages and sitepages. The sitepages was suppose to replace externalpages but it wasn't ready for 1.8. I agree with Brett on the names. The only reference that I could find was the listing name from the manifest. I'll update this and then close the ticket.
comment:3 Changed 21 months ago by Cash Costello
- Resolution set to fixed
- Status changed from new to closed
Fixes #3804 using Site Pages for the plugin name
Changeset: 26feaf21f61eb7337f698bc96ff416276c2bd504
comment:4 Changed 19 months ago by Cash Costello
Fixes #3804 using Site Pages for the plugin name
Changeset: 26feaf21f61eb7337f698bc96ff416276c2bd504

IIRC at some point site pages and external pages were separate plugins and we merged them or dropped one.
I don't like the name "External Pages" because I don't think it conveys the purpose of the plugin. We're stuck with the plugin id (directory name) for 1.8, but we do have the option to change all references. For 1.8.1 we should at least make sure the user-facing references are all the same.
We can also move the menu entry to the plugin settings menu.