Refactor abstractfilelisting for readability

Reason for the change

• Use QHash::keys() instead of re-implementing this logic
• Use contains function to make code a little easier to read
• Squash two loops into one by removing unnecessary temporary container
• Use struct instead of QPair to give values more meaningful names than first and second

Test plan

Run autotests