This is an automated email from the ASF dual-hosted git repository.

jpirek pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/netbeans-tools.git


The following commit(s) were added to refs/heads/master by this push:
     new aa213f7  [NETBEANS-3823] last_updated_at is explicitly handled in code 
and should not be auto updated
     new 394dc17  Merge pull request #34 from 
matthiasblaesing/pp3-last-updated-always
aa213f7 is described below

commit aa213f764b0b282b23b3d646988878b13fa6bb2d
Author: Matthias Bläsing <mblaes...@doppel-helix.eu>
AuthorDate: Tue Oct 27 19:08:42 2020 +0100

    [NETBEANS-3823] last_updated_at is explicitly handled in code and should 
not be auto updated
    
    The automatic update of the last_update_at timestamp creates irritating
    user experience. The plugin table hold a download counter, so every
    download is an update to the plugin row and will trigger the auto update
    of the timestamp, which is not the timestamp the user would expect.
---
 pp3/config/pp3.sql | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pp3/config/pp3.sql b/pp3/config/pp3.sql
index fe0e7a2..d58d339 100644
--- a/pp3/config/pp3.sql
+++ b/pp3/config/pp3.sql
@@ -68,7 +68,7 @@ CREATE TABLE IF NOT EXISTS `plugin` (
   `license` varchar(255) COLLATE utf8_czech_ci DEFAULT NULL,
   `author_id` int(11) NOT NULL REFERENCES user(id),
   `added_at` datetime DEFAULT CURRENT_TIMESTAMP,
-  `last_updated_at` datetime DEFAULT CURRENT_TIMESTAMP ON UPDATE 
CURRENT_TIMESTAMP,
+  `last_updated_at` datetime DEFAULT CURRENT_TIMESTAMP,
   `approved_at` datetime DEFAULT NULL,
   `url` varchar(255) COLLATE utf8_czech_ci DEFAULT NULL,
   `status` int(11) DEFAULT NULL,


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@netbeans.apache.org
For additional commands, e-mail: commits-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to