Opened 6 years ago
Closed 4 years ago
#3740 closed defect (bug) (reported-upstream)
WordCamp.org: cancelled camps visible with URL
Reported by: | sippis | Owned by: | sippis |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | WordCamp Site & Plugins | Keywords: | good-first-bug has-patch needs-testing |
Cc: |
Description
For example, WC Rhode Island (post ID 3057593) is marked as "Cancelled", but still visible when accessing the URL https://central.wordcamp.org/wordcamps/wordcamp-ri-usa/. It shouldn't since it is canceled, right?
Attachments (3)
Change History (11)
This ticket was mentioned in Slack in #meta-wordcamp by sippis. View the logs.
6 years ago
#2
@
6 years ago
- Keywords needs-patch good-first-bug added
- Owner set to sippis
- Status changed from new to assigned
#4
@
6 years ago
- Keywords has-patch needs-testing added; needs-patch removed
This was a somewhat larger issue than I expected. Looks like you can access to every WordCamp info page on central.wordcamp.org regardless of its status.
Does every old WordCamp have a status? In WME WC's didn't have any status so also valid old camps did start giving a 404. A quick look on the production version WC list gives a feeling that every camp has a status, but this needs to be double-checked before applying the patch to production.
Patch extends existing query_public_statuses_on_archives
function in wcpt
plugin to limit singular WordCamp views on central.wordcamp.org/wordcamps/* to view only camps with public status. I extended the existing function to check also singular views, because it made more sense than adding a new check to new function.
This ticket was mentioned in Slack in #meta-wordcamp by sippis. View the logs.
6 years ago
#6
@
5 years ago
Updated diff:
- renamed function
query_public_statuses_on_archives
toquery_public_statuses
for consisteny - better inline documentation
This ticket was mentioned in Slack in #meta-wordcamp by iandunn. View the logs.
5 years ago
#8
@
4 years ago
- Resolution set to reported-upstream
- Status changed from assigned to closed
This ticket has been moved to GitHub https://github.com/WordPress/wordcamp.org/issues/602
Today in WC bug scrub Ian said (note ro me for doing patch)