Looks good, thanks 👍
Could you please add a line with your name in https://github.com/FreshRSS/FreshRSS/blob/dev/CREDITS.md
and a line in the changelog? https://github.com/FreshRSS/FreshRSS/blob/dev/CHANGELOG.md

P.S.: @Trim, putting your name in the Credits file was just a suggestion; no problem if you do not want :-)

Oh, I didn't see your first comment, I'll add a commit to this PR.

Note, that I tried to run this modification on my server and I run into a related issue: I have an RSS feed containing two articles with same URI: http://www.geany.org/Main/HomePage?action=blogrss contains two articles with URI http://www.geany.org/Main/20170717 (the latest two).

This issue continue to rise a PostgreSQL error, but it didn't block all the transaction:

2017-08-15 14:15:22 CEST [4159-2] freshrss@freshrss DÉTAIL:  La clé « (id_feed, guid)=(67, http://www.geany.org/Main/20170717) » existe déjà.
2017-08-15 14:15:22 CEST [4159-3] freshrss@freshrss INSTRUCTION :  INSERT INTO "adrien_entrytmp" (id, guid, title, author, content, link, date, "lastSeen", hash, is_read, is_favorite, id_feed, tags) VALUES($1, $2, $3, $4, $5, $6, $7, $8, decode($9, 'hex'), $10, $11, $12, $13)
2017-08-15 14:15:22 CEST [4159-4] freshrss@freshrss ERREUR:  la transaction est annulée, les commandes sont ignorées jusqu'à la fin du bloc
	de la transaction

Now, I don't know two thing:

• why not all the block contained inside the BEGIN END block has been reverted (other feeds have been updated)
• how FreshRSS should handle articles with same URI

Thanks for the other related issue. For the time being, I would suggest to add the same WHERE clause than the one you have made, for the initial insert into the temporary table.
It is not a problem to have entries with identical URL, IF they provide a GUID or equivalent (e.g. rdf:about). When they do not provide a GUID, then it is not trivial to make a good unique identifier (hashing is possible, but then it fails when articles are updated). So the behaviour right now should be to keep the newest article of a given URL when there is no GUID.

I've just understood the log message with your comment:
the error I see now is not inside the commitNewEntries, but inside the addEntry function.

So this pull request still make sense and I still understand transactions :)

PS: I updated changelog and contributors files

Indeed, during the discovery of new articles (that is when articles are added to the temporary table), there is one transaction by feed. Then, when committing to the main table, there is one single transaction for all new articles.

@Trim Could you please open a new issue for the related problem you reported above? #1613 (comment)