Making WordPress.org

Opened 11 years ago

Closed 10 years ago

#246 closed defect (bug) (fixed)

Theme Diffs occasionally diff wrong versions

Reported by: nofearinc's profile nofearinc Owned by:
Milestone: Priority: normal
Component: Theme Directory Keywords:
Cc:

Description

While conducting theme reviews today, some of the themes that updated versions with more than 1 step (like from 2.08 to 2.11) had their diffs wrong.

For instance, iRibbon 2.0.22 was uploaded after 2.0.20, but the diff link is:

http://themes.trac.wordpress.org/changeset?old_path=/iribbon/2.0.20&new_path=/iribbon/2.0.21

Another strange thing is that 2.0.21 is the "new path" while 2.0.22 is the theme version in Trac, whilst style.css defines Version: 2.0.21.

Change History (4)

#1 @nacin
11 years ago

  • Component changed from Trac to Themes Directory

Everything appears to be good here. What happened is the current version was 2.0.20 and then they uploaded 2.0.21. That created the initial ticket.

Before 2.0.21 was approved, 2.0.22 was uploaded. That caused the first comment, which renamed the ticket to 2.0.22 (but kept the description intact).

It's probable that the comprehensive diff from 2.0.20 to 2.0.22 should be linked from somewhere, that way a theme reviewer doesn't accidentally just review the diff from 2.0.21 to 2.0.22, ignoring the changes that occurred between 2.0.20 and 2.0.21.

Changing components as this has to do with the theme directory upload code, not Trac itself.

#2 @nofearinc
11 years ago

Thanks for the feedback nacin, that makes sense. The only problem (as you mentioned yourself) is that reviewers would see (by default) the diff with the previous uploaded version, and not previously approved (and there could be several uploads there that are not visible in the list of reviewed themes).

This ticket was mentioned in IRC in #wordpress-meta by sams. View the logs.


11 years ago

#4 @obenland
10 years ago

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