Merged #42 "Fix failure of RedisTicketService to determine hasTicket"

James Moger

2014-04-08 8fbf6fb68b0528d55114865fbb869a0a39709873

Merged #42 "Fix failure of RedisTicketService to determine hasTicket"

 releases.moxie

@@ -14,6 +14,7 @@
    - Fix intermittent NPE in determining commit date in RefModel (issue-401)
    - Fix closing ticket on push by parsing commit messages for closes|fixes (issue-404)
    - Ensure the Lucene ticket index is updated on repository deletion.
    - Fixed failure to properly determine hasTicket in RedisTicketService
    changes:
    - Specify the --dailyLogFile option for the Ubuntu and CentOS service scripts (issue-348)
    - Improve logging for missing LDAP uid attribute when synchronizing (issue-394)

 src/main/java/com/gitblit/tickets/RedisTicketService.java

@@ -168,7 +168,7 @@
        }
        try {
            Boolean exists = jedis.exists(key(repository, KeyType.journal, ticketId));
            return exists != null && !exists;
            return exists != null && exists;
        } catch (JedisException e) {
            log.error("failed to check hasTicket from Redis @ " + getUrl(), e);
            pool.returnBrokenResource(jedis);