[Owncloud] Review Request: Fix bug: filenames in filelisting are not shown for files without extension
Elias Probst
mail at eliasprobst.eu
Tue Aug 30 18:20:16 UTC 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102492/
-----------------------------------------------------------
(Updated Aug. 30, 2011, 6:20 p.m.)
Review request for ownCloud.
Changes
-------
Added a missing . in front of the extension.
Summary
-------
There is a bug in the filelisting (files/index.php) which causes troubles when listing files which don't have an extension, e.g.
Test123 vs. Test123.txt
The filenames of such files were only shown when hovering them.
The reason for this behaviour was the hackish approach on splitting the filename by using substr() and strrpos().
The bugfix uses now PHP's builtin pathinfo() to do this job.
Diffs (updated)
-----
files/index.php 0595463
Diff: http://git.reviewboard.kde.org/r/102492/diff
Testing
-------
Screenshots
-----------
Filename for files without extension is only shown on hover
http://git.reviewboard.kde.org/r/102492/s/244/
Thanks,
Elias
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/owncloud/attachments/20110830/7983483c/attachment.html>
More information about the Owncloud
mailing list