otwstephanie / otwarchive3

Clone of OTWarchive, for merge testing
0 stars 0 forks source link

Sorting stories by title has strange results #310

Closed otwstephanie closed 10 years ago

otwstephanie commented 10 years ago

From dod...@yahoo.com on September 07, 2008 03:58:49

What archive revision are you testing on? R585 What steps will reproduce the problem? 1. go to 'view all stories' page

  1. Underneath 'view all stories' title there's an option to sort work by title. Click on that What is the expected output? What do you see instead? I expected to see works alphabetically from A-Z. What happens is strange. It's alphabetical but
    it doesn't start at 'A'. Here's the listing that came up for me:

Morgan's Dog, Mulga Bill's Bicycle, Old pardon, the son of reprieve, sunrise on the coast, the man from ironbark, waltzing matilda, a bush christening, adfgafg, all your base were deleted, apple, arrrgh...and so on, the rest being alphabetical as far as I could see.

But don't count on it being in that order when you try it because I kept clicking on sort by title and a different story order kept showing up.

Original issue: http://code.google.com/p/otwarchive/issues/detail?id=310

otwstephanie commented 10 years ago

From Black0Sa...@gmail.com on September 08, 2008 06:03:44

Labels: Roadmap-MainNavigation

otwstephanie commented 10 years ago

From shal...@gmail.com on September 11, 2008 10:18:41

Status: Accepted
Labels: Milestone-Beta

otwstephanie commented 10 years ago

From bookk...@gmail.com on September 13, 2008 12:10:02

(I'm working off of http://archiveofourown.org/en/users/ice_is_blue/works ) Revision 677 Here's what I noticed is happening...

When you first get to that works page, your options and links are:

Title Sort = http://archiveofourown.org/en/users/ice_is_blue/works?sort_column=title&sort_direction=ASC Date Sort = http://archiveofourown.org/en/users/ice_is_blue/works?sort_column=date&sort_direction=ASC Word Count Sort = http://archiveofourown.org/en/users/ice_is_blue/works?sort_column=word_count&sort_direction=ASC If a user clicks on ANY of those sorting links ALL links automatically change to:

Title Sort = http://archiveofourown.org/en/users/ice_is_blue/works?sort_column=title&sort_direction=DESC Date Sort = http://archiveofourown.org/en/users/ice_is_blue/works?sort_column=date&sort_direction=DESC Word Count Sort = http://archiveofourown.org/en/users/ice_is_blue/works?sort_column=word_count&sort_direction=DESC So....

It looks like the Default sort (on this page, at least) is Date, DESC. (So maybe the first sorting option should be date?)

Anywhoo, maybe something like:

Date [up arrow] [down arrow] | Title [up] [down] | Word Count [up] [down]

OR... just not allow ascending and descending for each sorting category. Ascending only makes sense for both Title and Word Count. Date, I could accept either way, or even a toggle... but as it currently stands... sort results are unpredictable.

otwstephanie commented 10 years ago

From amb...@gmail.com on September 18, 2008 00:21:04

the original problem with it starting at M was because of leading spaces in the titles. this was fixed in r755 the second problem from bookkish has not been fixed

otwstephanie commented 10 years ago

From shal...@gmail.com on September 30, 2008 23:23:27

Fixed as of rev 887.

Status: FixedButUnverified
Owner: shalott

otwstephanie commented 10 years ago

From dod...@yahoo.com on October 01, 2008 01:18:12

As of r888 it still isn't showing the titles properly for me. I've attached a screenshot of the first 5 stories that show up. After that it seems to display properly

Attachment: Picture 6.png

otwstephanie commented 10 years ago

From shal...@gmail.com on October 01, 2008 17:04:50

The sorting issue was due to bad data in the db -- works/chapters with leading spaces. rev 901 includes a migration to eliminate all of these.

Sorting on titles should now work fine both on search results and filtering results, and should skip leading articles ('a', 'an', 'the'). Need to think about how this will work for non-english works, but that will be for Another Day.

otwstephanie commented 10 years ago

From dod...@yahoo.com on October 01, 2008 17:16:31

Sorry but I just checked and there's still one story showing up out of order:( 'Life Without Fraser or How Ray Learned to Stop Worrying and Love the Moose' shows up first.

Screenshot attached and it's in R901

Attachment: Picture 5.png

otwstephanie commented 10 years ago

From shal...@gmail.com on October 01, 2008 20:28:22

Yep, there were works without required tags. New migrations in rev 905 should have now laid to rest once and for all.

otwstephanie commented 10 years ago

From dod...@yahoo.com on October 01, 2008 20:33:47

Yay\o/ FixedandVerified!

Status: FixedAndVerified

otwstephanie commented 10 years ago

From amb...@gmail.com on February 24, 2009 17:43:40

Status: DeployedToBeta

otwstephanie commented 10 years ago

From Black0Sa...@gmail.com on February 24, 2009 17:43:44

Updating status options

otwstephanie commented 10 years ago

From Black0Sa...@gmail.com on February 24, 2009 19:23:12

Updating status on discussion with Jennifew and Sidra

Status: VerifiedOnBeta

otwstephanie commented 10 years ago

From autu...@gmail.com on November 13, 2010 14:14:03

listing all former issues for automated tests

Status: NeedsTests

otwstephanie commented 10 years ago

From autu...@gmail.com on November 14, 2010 11:40:28

Converting from NeedsTest status to NeedsTest label

Status: VerifiedOnBeta
Labels: NeedsTests