Revert "TN-3324 robust handling of unsortable patient list attributes"

[pbugni]

Reverts #4414

Wrong target. Forgot to change base after we decided this didn't need to be a hotfix.

[pep8speaks]

Hello @pbugni! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file tests/integration_tests/test_login.py:

Line 40:101: E501 line too long (144 > 100 characters)
Line 41:101: E501 line too long (144 > 100 characters)
Line 50:101: E501 line too long (140 > 100 characters)
Line 60:101: E501 line too long (150 > 100 characters)
Line 61:101: E501 line too long (166 > 100 characters)

[pbugni]

@achen2401 can you review these changes.

I unintentionally merged this into master, as I'd opened the PR when I thought we'd hotfix the preferences issue. The majority of changes come from cleaning up broken test issues that came up out of nowhere.

I ask, as I see additional front end files I didn't have in my PR, so I'm wondering if those should be included in this attempt to roll master back to current prod?

[achen2401]

@pbugni Can you rebase this against the develop branch? I see that some of the frontend changes I made aren’t there or reverted.

[achen2401]

@pbugni please ignore my previous comment. I see this is reverting the commit from the master branch. I believe the frontend changes I made were correctly committed to the develop branch (they aren’t in the master branch yet)