Commit Graph
1357 Commits
Author SHA1 Message Date
Alexandre Alapetite 73bbdaa015 Merge pull request #542 from aledeg/sqlite-stats
Add statistics support for Sqlite
2014-07-20 15:39:06 +02:00
Alexis Degrugillier 68c0a827d2 Add statistics support for Sqlite
Add statistics support for Sqlite by tweeking one query and rewrite an other. The rewrite implied a complete refactor of the MySql query as well.
Now the code is more flexible and make less queries to the database.

See #527
2014-07-18 20:35:03 -04:00
Alexandre Alapetite b466b6075e Merge pull request #535 from aledeg/confirm
Add a confirmation when using shortcuts
2014-07-17 12:07:30 +02:00
Alexandre Alapetite 3403af3050 SQL: Bug with order ASC
https://github.com/marienfressinaud/FreshRSS/issues/538
2014-07-17 11:56:15 +02:00
Alexandre Alapetite 5d7deb6d25 CSS: Bug dancing titles
https://github.com/marienfressinaud/FreshRSS/issues/534
Tested on Firefox 31 (Win8), Firefox 30 (Ubuntu), IE11, Chrome 36
2014-07-14 19:35:31 +02:00
Alexandre Alapetite 5a7e602005 Bug main view prefix title
https://github.com/marienfressinaud/FreshRSS/issues/536
2014-07-14 00:30:55 +02:00
Alexandre Alapetite fb65d7ee49 Bug compatibility prefix title
https://github.com/marienfressinaud/FreshRSS/issues/536
The previous syntax worked only in Firefox
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/default_parameters
2014-07-13 22:42:33 +02:00
Alexandre Alapetite 2da4c13263 Number of unread articles as prefix in page title
https://github.com/marienfressinaud/FreshRSS/issues/536
2014-07-13 22:36:34 +02:00
Alexis Degrugillier 32fca7c4e1 Add a confirmation when using shortcuts
Before, it was possible to mark everything as read with no confirmation using shortcut even if the configuration introduced in a previous commit asked for a confirmation.
Now, depending of the configuration, a confirm dialog box is displayed when requested in the configuration.
2014-07-12 09:00:26 -04:00
Marien Fressinaud 85bae5436c Fix bug changement default view
- It was impossible to set default view to "Show all articles"
- "0" was compared to 0 ("O" === 0 => false)
- cast value from Minz_Request into integer
2014-07-10 20:44:35 +02:00
Marien Fressinaud 72b6bf8d88 Fix header title alignment and change color of Flat icon 2014-07-10 19:37:19 +02:00
Marien Fressinaud 0878b32faf Fix themes
- Add padding for "Connection" button
- Adjust height of .header and #global. I don't really like this solution
  but I can't see any other solution
2014-07-10 19:25:27 +02:00
Marien Fressinaud 52763ed97d Fix a bug during installation
- If MySQL database was configured and user choosed SQLite, an error was
  raised at the first submit.
- Cast STEP into integer
2014-07-10 18:46:38 +02:00
Marien Fressinaud f56dd400b7 Add base-theme in p/themes
This base theme is not visible by default (no name). You can use it
to create new theme.
Fix https://github.com/marienfressinaud/FreshRSS/issues/533 (comment)
2014-07-10 17:42:42 +02:00
Alexandre Alapetite 6c83264f83 SQL: MySQL speed optimisation
Would be nice to test with other versions of MySQL too.

Comparison before/after:

mysql> EXPLAIN SELECT e.id, e.guid, e.title, e.author,
UNCOMPRESS(content_bin) AS content, e.link, e.date, e.is_read,
e.is_favorite, e.id_feed, e.tags FROM `freshrss_alex_entry` e INNER JOIN
(SELECT e1.id FROM `freshrss_alex_entry` e1 INNER JOIN
`freshrss_alex_feed` f ON e1.id_feed=f.id WHERE f.priority > 0 AND
(e1.id >= 1381615200000000 OR e1.is_read=0 OR e1.is_favorite=1 OR
(f.keep_history <> 0)) ORDER BY e1.id DESC LIMIT 31) e2 ON e2.id=e.id
ORDER BY e.id DESC;

+----+-------------+------------+--------+-------------------------------------+----------+---------+---------------+------+----------------------------------------------+
| id | select_type | table      | type   | possible_keys
| key      | key_len | ref           | rows | Extra
|

+----+-------------+------------+--------+-------------------------------------+----------+---------+---------------+------+----------------------------------------------+
|  1 | PRIMARY     | <derived2> | ALL    | NULL
| NULL     | NULL    | NULL          |   31 | Using temporary; Using
filesort              |
|  1 | PRIMARY     | e          | eq_ref | PRIMARY
| PRIMARY  | 8       | e2.id         |    1 |
|
|  2 | DERIVED     | f          | range  | PRIMARY,priority,keep_history
| priority | 1       | NULL          |  137 | Using where; Using
temporary; Using filesort |
|  2 | DERIVED     | e1         | ref    |
PRIMARY,id_feed,is_favorite,is_read | id_feed  | 2       | freshrss.f.id
|  452 | Using where                                  |

+----+-------------+------------+--------+-------------------------------------+----------+---------+---------------+------+----------------------------------------------+
4 rows in set (3.50 sec)

mysql> EXPLAIN SELECT e.id, e.guid, e.title, e.author,
UNCOMPRESS(content_bin) AS content, e.link, e.date, e.is_read,
e.is_favorite, e.id_feed, e.tags FROM `freshrss_alex_entry` e INNER JOIN
(SELECT e1.id FROM `freshrss_alex_entry` e1 INNER JOIN
`freshrss_alex_feed` f ON e1.id_feed=f.id WHERE f.priority > 0 AND e1.id
<=9000000000000000 AND (e1.id >= 1381615200000000 OR e1.is_read=0 OR
e1.is_favorite=1 OR (f.keep_history <> 0)) ORDER BY e1.id DESC LIMIT 31)
e2 ON e2.id=e.id ORDER BY e.id DESC;

+----+-------------+------------+--------+-------------------------------------+---------+---------+---------------------+-------+---------------------------------+
| id | select_type | table      | type   | possible_keys
| key     | key_len | ref                 | rows  | Extra
|

+----+-------------+------------+--------+-------------------------------------+---------+---------+---------------------+-------+---------------------------------+
|  1 | PRIMARY     | <derived2> | ALL    | NULL
| NULL    | NULL    | NULL                |    31 | Using temporary;
Using filesort |
|  1 | PRIMARY     | e          | eq_ref | PRIMARY
| PRIMARY | 8       | e2.id               |     1 |
|
|  2 | DERIVED     | e1         | range  |
PRIMARY,id_feed,is_favorite,is_read | PRIMARY | 8       | NULL
| 70084 | Using where                     |
|  2 | DERIVED     | f          | eq_ref | PRIMARY,priority,keep_history
| PRIMARY | 2       | freshrss.e1.id_feed |     1 | Using where
|

+----+-------------+------------+--------+-------------------------------------+---------+---------+---------------------+-------+---------------------------------+
4 rows in set (0.01 sec)
2014-07-10 15:49:51 +02:00
Marien Fressinaud ab9d656135 Block user to click many times on the refresh button
See https://github.com/marienfressinaud/FreshRSS/issues/513
2014-07-09 20:20:17 +02:00
Marien Fressinaud c1728fe0d7 Coding style header.phtml + possibility to change logo
See https://github.com/marienfressinaud/FreshRSS/issues/532
2014-07-09 19:39:21 +02:00
Marien Fressinaud 62dea24777 Fix Dark theme
See https://github.com/marienfressinaud/FreshRSS/issues/532
2014-07-09 19:28:58 +02:00
Marien Fressinaud 0a51ac6033 Fix Dark theme + Changelog 2014-07-08 23:35:59 +02:00
Marien Fressinaud 61decd4528 Dark design based on default template
Need reviews

See https://github.com/marienfressinaud/FreshRSS/issues/320
2014-07-08 23:32:54 +02:00
Marien Fressinaud 1b140d6c06 Merge branch 'dev' of github.com:marienfressinaud/freshrss into dev 2014-07-08 21:32:39 +02:00
Marien Fressinaud 8b3ec25218 Fix Flat design 2014-07-08 21:32:25 +02:00
Alexandre Alapetite 0f842c1aea SQL: removed superfluous transactions to avoid some dead locks 2014-07-08 14:14:02 +02:00
Alexandre Alapetite 436f9a432b Merge pull request #531 from aledeg/issue519
Add a confirmation dialog when clicking on the big tick
2014-07-08 12:30:18 +02:00
Alexis Degrugillier 21d8c03ac9 Add a configuration parameter
Add a parameter to choose wether or not the user want to display a confirmation dialog when clicking on "mark all as read" actions.
2014-07-07 19:34:47 -04:00
Alexis Degrugillier 92f1d6d658 Add confirmation on button as well 2014-07-07 19:34:38 -04:00
Alexis Degrugillier 41c2966543 Add a confirmation dialog when clicking on the big tick
See #519
Before, when clicking on the big tick to mark all articles as read, it triggers directly the action.
Now, it asks for a confirmation to block unwanted user clicks.
2014-07-07 19:34:17 -04:00
Marien Fressinaud cdc8262c93 Changelog Flat design and user queries 2014-07-07 22:08:42 +02:00
Marien Fressinaud 7a653e7368 Merge branch 'dev' of github.com:marienfressinaud/freshrss into dev 2014-07-07 22:01:53 +02:00
Marien Fressinaud dd62729228 Flat design based on default template
- stats are not terminated yet
- need reviews

See https://github.com/marienfressinaud/FreshRSS/issues/320
2014-07-07 21:58:34 +02:00
Alexandre Alapetite e358c5e2e1 SQL: SQLite syntax uniformisation
https://github.com/marienfressinaud/FreshRSS/commit/805c91da98c2f582e279f3c853fba9e43f572419#diff-101042bec0ff3ac9d691b2e77fca3313R7
2014-07-07 21:12:13 +02:00
Alexandre Alapetite 439a0e2991 SQL: improved performance for adding new articles 2014-07-07 18:25:48 +02:00
Alexandre Alapetite 3bbd0e446f Prepare statistics for SQLite
Temporarily disable 30-day statistics for SQLite
https://github.com/marienfressinaud/FreshRSS/issues/100
https://github.com/marienfressinaud/FreshRSS/issues/90
2014-07-06 11:54:00 +02:00
Alexandre Alapetite 96057fe491 Minor #524
https://github.com/marienfressinaud/FreshRSS/issues/524
2014-07-06 10:33:49 +02:00
Alexandre Alapetite 6a595ac0ad Merge pull request #525 from aledeg/issue524
Change behavior when selecting an article
2014-07-06 10:28:39 +02:00
Alexandre Alapetite 237be11426 Changelog TTL
https://github.com/marienfressinaud/FreshRSS/issues/250
2014-07-06 10:26:23 +02:00
Alexandre Alapetite 3d3b33ea2e Bug global TTL
https://github.com/marienfressinaud/FreshRSS/issues/250
2014-07-06 10:12:37 +02:00
Alexandre Alapetite 72924cb5fe Minor white space parentheses 2014-07-06 00:10:58 +02:00
Alexandre Alapetite bc8eb560af Add TTL to control feed freshness
https://github.com/marienfressinaud/FreshRSS/issues/250
2014-07-06 00:09:31 +02:00
Alexandre Alapetite b48dc25963 Merge pull request #516 from aledeg/stats
Refactor statistics
2014-07-05 18:43:35 +02:00
Alexis Degrugillier 6b900fe31b Change behavior when selecting an article
See #524
Before, when the configuration was set so the article is marked as read when selected, if you mark it as unread while it is open and then close it, it is marked again as read. This behavior was not straightforward.
Now, the article is not marked again as read.
2014-07-05 09:06:45 -04:00
Alexandre Alapetite 76b64a6907 Bug search
Following introduction of user-queries
https://github.com/marienfressinaud/FreshRSS/commit/2b8dc666345d334f50bf2f4f32f0b127edb40c3a#diff-be03dd1f6b53ebb2682ba9dd334bc273
2014-07-05 12:08:17 +02:00
Alexandre Alapetite d632900b2f Autoload more articles if needed on big screens 2014-07-05 11:46:29 +02:00
Alexandre Alapetite b8e6a3b5b2 Bug feed->hash for 301 redirections, favicons 2014-07-05 11:17:23 +02:00
Alexandre Alapetite 805c91da98 Add support for SQLite
https://github.com/marienfressinaud/FreshRSS/issues/100
Warning: MySQL has been changed too, so bugs may have been introduced
2014-07-05 01:52:41 +02:00
Alexandre Alapetite 1b43167fb8 SQL: Add f.ttl column to control feed cache duration
Preparation of https://github.com/marienfressinaud/FreshRSS/issues/250
Will also be used to disable automatic update for selected feeds
2014-07-05 01:20:45 +02:00
Alexandre Alapetite b34f59e85a Preparation #3 for SQLite
https://github.com/marienfressinaud/FreshRSS/issues/100
2014-07-03 22:48:29 +02:00
Alexandre Alapetite 2501bb337e Preparation #2 for SQLite
https://github.com/marienfressinaud/FreshRSS/issues/100
2014-07-03 22:11:25 +02:00
Alexandre Alapetite d6f4141086 Preparation for SQLite
https://github.com/marienfressinaud/FreshRSS/issues/100
2014-07-03 21:26:30 +02:00
Alexandre Alapetite 8a5050289e New install logic
Now the install.php is kept in ./app/ and the trigger is
./data/do-install.php
There is no need anymore to have write access in ./p/i/
2014-07-03 21:20:11 +02:00