Merge branch 'develop' into update_travis_tests

CHANGELOG.md

@@ -7,6 +7,7 @@ Exclamation symbols (:exclamation:) note something of importance e.g. breaking c
 ### Added
 ### Changed
 - Updated Travis to use Trusty containers (for HHVM) and add PHP 7.2 to the tests.
+- Add debug log entry instead of throwing an exception for duplicate updates.
 ### Deprecated
 ### Removed
 ### Fixed

CREDITS

@@ -23,6 +23,5 @@ W: http://noplanman.ch
 D: Maintainer and Collaborator
 
 N: Jack'lul (alias)
-E: jacklul@jacklul.com
-W: http://jacklul.com
+E: jacklulcat@gmail.com
 D: Maintainer and Collaborator

src/DB.php

@@ -492,10 +492,6 @@ class DB
         $update_id   = $update->getUpdateId();
         $update_type = $update->getUpdateType();
 
-        if (count(self::selectTelegramUpdate(1, $update_id)) === 1) {
-            throw new TelegramException('Duplicate update received!');
-        }
-
         // @todo Make this simpler: if ($message = $update->getMessage()) ...
         if ($update_type === 'message') {
             $message = $update->getMessage();

src/Telegram.php

@@ -457,6 +457,13 @@ class Telegram
         //This is necessary to "require" all the necessary command files!
         $this->getCommandsList();
 
+        //Make sure we don't try to process update that was already processed
+        $last_id = DB::selectTelegramUpdate(1, $this->update->getUpdateId());
+        if ($last_id && count($last_id) === 1) {
+            TelegramLog::debug('Duplicate update received, processing aborted!');
+            return Request::emptyResponse();
+        }
+
         DB::insertRequest($this->update);
 
         return $this->executeCommand($command);