Review Request 128942: Partial implementation of Bug 358075. Added "Accessed" column, and renamed "Date" to "Modified" as requested by several tickets.

[Don Nguyen]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128942/
-----------------------------------------------------------

(Updated Sept. 19, 2016, 4:42 a.m.)


Review request for Dolphin.


Bugs: 358075
    https://bugs.kde.org/show_bug.cgi?id=358075


Repository: dolphin


Description (updated)
-------

Several tickets want additional file time fields to be displayed such as creation time, access time, etc.   They also want "Date" to be renamed to something more meaningful.  What I've implemented was to add an "Accessed" column to show the file access time.  Creation time is not stored in Linux so displaying this is not possible.  I renamed the "Date" column to "Modified".  I implemented this in code by adding a new "accesstime" and AccessTimeRole to KFileItemsModel.  I changed the dateRoleGroups function to timeRoleGroups and it now takes a parameter to distinguish which type of time(access or modification) it's dealing with.  I also changed the DateRole enum to ModificationTimeRole so that it makes more sense with the new addition.  
I am new to KDE and open source development.  This is my first attempt at working with ReviewBoard.


Diffs
-----

  src/kitemviews/kfileitemlistwidget.cpp 7e5b4bb 
  src/kitemviews/kfileitemmodel.h 8a0df72 
  src/kitemviews/kfileitemmodel.cpp 1f94972 

Diff: https://git.reviewboard.kde.org/r/128942/diff/


Testing
-------

Tested adding new "Accessed" column and verified data shown is correct.  Tested group sorting by Accessed time.  Tested "Modified" column still works as before when it was named "Date".


Thanks,

Don Nguyen

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20160919/3298ba9f/attachment.htm>