Defect #7099

404 when entering time with blank issue #

Added by Adam Soltys about 7 years ago. Updated about 7 years ago.

Status:ClosedStart date:2010-12-13
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:Time tracking
Target version:-
Resolution:Fixed Affected version:

Description

When entering time against a project without specifying an issue number, I get a 404 error when running on the latest edge version of the code.

The problem seems to have been introduced by r4461: "Adds REST API for TimeEntries (#6823)."

I've been trying to figure out how to solve it but haven't managed to yet.

fix_time_entry_404.diff Magnifier (2.38 KB) Adam Soltys, 2010-12-13 07:41

fix_time_entry_404.diff Magnifier (2.34 KB) Adam Soltys, 2010-12-13 07:52

Associated revisions

Revision 4511
Added by Jean-Philippe Lang about 7 years ago

Fixed: 404 when entering time with blank issue id (#7099).

History

#1 Updated by Adam Soltys about 7 years ago

I think I've figured it out. Attaching patch with failing test case and fix. Please review.

#2 Updated by Adam Soltys about 7 years ago

Sorry, that test wouldn't actually fail before the fix. Now it should.

#3 Updated by Jean-Philippe Lang about 7 years ago

  • Status changed from New to Closed
  • Resolution set to Fixed

Fixed in r4511, thanks for pointing this out.

Also available in: Atom PDF