WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#26565 closed task (blessed) (fixed)

revisions.js routing doesn't actually do anything. Remove dead code.

Reported by: markjaquith's profile markjaquith Owned by: nacin's profile nacin
Milestone: 3.9 Priority: low
Severity: trivial Version: 3.6
Component: Revisions Keywords: has-patch dev-feedback needs-testing
Focuses: Cc:

Description

matveb and adamsilverstein pointed out to me that the revisions.js routing doesn't actually do anything because Backbone doesn't support parsing of query args. The reason that it works is that we're pre-populating the variables in PHP and handing them off to the JS. So the routes and their callback are dead code that can just be removed.

Attachments (1)

26565.diff​ (1.1 KB) - added by adamsilverstein 12 years ago.
remove unused router code

Download all attachments as: .zip

Change History (6)

@adamsilverstein
12 years ago

remove unused router code

#1 @adamsilverstein
12 years ago

  • Cc adamsilverstein@… added
  • Keywords has-patch dev-feedback added

26565.diff​​ removed routing code that is no longer used

#2 @nacin
12 years ago

  • Milestone changed from Awaiting Review to 3.9

#3 @adamsilverstein
12 years ago

  • Keywords needs-testing added

I tested and would appreciate anyone who can pitch in here with a sanity check to double check this change doesn't break anything unexpectedly, throw console errors, etc.

#4 @kovshenin
12 years ago

Looks sane. Tested in Chrome and Firefox on OS X.

#5 @nacin
12 years ago

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

In 27273:

Remove unused routing code in revisions.

props adamsilverstein.
fixes #26565.

Note: See TracTickets for help on using tickets.