thomasklein / redmine2mite

Redmine2mite connects your Redmine account with your mite.account. Track your time easily on issues within Redmine and get them automatically send to mite.
http://mite.yo.lk/
MIT License
13 stars 6 forks source link

Fixed bug in MiteSynchronizer::TimeEntries with more than 500 entries #28

Closed yolk closed 11 years ago

yolk commented 11 years ago

Hi Thomas,

ein mite-Kunde hatte sich gestern bei uns gemeldet, dass redmine2mite Zeiteinträge in mite beim Synchronisieren fälschlicherweise gelöscht hätte.

Ich bin mal in denen Code getaucht und habe einen Fehler in MiteSynchronizer::TimeEntries#remote_records gefunden: Dort wurden alle Einträge nach den ersten 500 ignoriert.

Anbei ein Fix für das Problem. Wäre super wenn du recht schnell 2.1.1 rausbringen könntest.

Danke & ein schönes Wochenende, Sebastian.

thomasklein commented 11 years ago

Hi Sebastian!

Vielen Dank für den Bugfix! Merged und tagged: https://github.com/thomasklein/redmine2mite/releases/tag/2.1.1

Dir auch noch ein schönes WE!

Thomas

Am 30.08.2013 um 14:20 schrieb Sebastian Munz notifications@github.com:

Hi Thomas,

ein mite-Kunde hatte sich gestern bei uns gemeldet, dass redmine2mite Zeiteinträge in mite beim Synchronisieren fälschlicherweise gelöscht hätte.

Ich bin mal in denen Code getaucht und habe einen Fehler in MiteSynchronizer::TimeEntries#remote_records gefunden: Dort wurden alle Einträge nach den ersten 500 ignoriert.

Anbei ein Fix für das Problem. Wäre super wenn du recht schnell 2.1.1 rausbringen könntest.

Danke & ein schönes Wochenende, Sebastian.

You can merge this Pull Request by running

git pull https://github.com/yolk/redmine2mite master Or view, comment on, or merge it at:

https://github.com/thomasklein/redmine2mite/pull/28

Commit Summary

Fixed bug in MiteSynchronizer::TimeEntries with more than 500 entries File Changes

M lib/mite_synchronizers/time_entries.rb (5) Patch Links:

https://github.com/thomasklein/redmine2mite/pull/28.patch https://github.com/thomasklein/redmine2mite/pull/28.diff

yolk commented 11 years ago

Dankeschön Thomas für die schnelle Veröffentlichung!