Mailinglist Archive: opensuse-buildservice (256 mails)

< Previous Next >
Re: [opensuse-buildservice] Re: [PATCH] [api] remove obsolete or wrong FIXME comments. Move some to 3.0 release.
  • From: Adrian Schröter <adrian@xxxxxxx>
  • Date: Tue, 23 Aug 2011 14:18:52 +0200
  • Message-id: <3743578.L9ebe8WBEc@scherben>
Am Dienstag, 23. August 2011, 14:11:30 schrieb Sascha Peilicke:
On Tuesday 23 August 2011 13:40:27 you wrote:
From: Adrian Schröter <adrian@xxxxxxx>

--- a/src/api/app/controllers/request_controller.rb
+++ b/src/api/app/controllers/request_controller.rb
@@ -50,8 +50,6 @@ class RequestController < ApplicationController

end

# Filter by request type (submit, delete, ...)

- #FIXME/FIXME2.3: This should be params[:type] instead but for
whatever reason, all - # webui controllers already set params[:type]
to 'request' (always).

Is that issue solved already?

We decided to use "types" here, because it supports the multiple items
notation (similar to "states", "reviewstates"). As a side effect we don't
have the conflict with "type" anymore.

There is a different place where we would like to use "type", I renamed it
there as FIXME3.0, so we can re-evaluate with Rails 3. And we can not change
the API before 3.0 anyway.

--
Adrian Schroeter
SUSE Linux Products GmbH
email: adrian@xxxxxxx

--
To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: opensuse-buildservice+help@xxxxxxxxxxxx

< Previous Next >
Follow Ups
References