should ?t=rename be deprecated in favour of ?t=move ? #1930

Open
opened 2013-03-14 17:54:03 +00:00 by davidsarah · 3 comments
davidsarah commented 2013-03-14 17:54:03 +00:00
Owner

In #1732 we added a ?t=move operation to the web-API, which is strictly more general than ?t=rename. Should we deprecate the latter?

(I tend towards keeping ?t=rename.)

In #1732 we added a `?t=move` operation to the web-API, which is strictly more general than `?t=rename`. Should we deprecate the latter? (I tend towards keeping `?t=rename`.)
tahoe-lafs added the
code-frontend-web
normal
defect
1.9.2
labels 2013-03-14 17:54:03 +00:00
tahoe-lafs added this to the soon milestone 2013-03-14 17:54:03 +00:00
zooko commented 2013-03-14 17:57:10 +00:00
Author
Owner

I tend towards deprecating ?t=rename, because I like minimalism in an API and I think of every API function as another thing that slows down our progress because we have to support it. So I'm about +½ on deprecating it at this point.

I tend towards deprecating `?t=rename`, because I like minimalism in an API and I think of every API function as another thing that slows down our progress because we have to support it. So I'm about +½ on deprecating it at this point.
davidsarah commented 2013-03-15 05:34:31 +00:00
Author
Owner

Well, I suppose we can only deprecate it after we've changed tahoe mv to stop using it, in any case.

Well, I suppose we can only deprecate it after we've changed `tahoe mv` to stop using it, in any case.
davidsarah commented 2013-03-15 05:37:42 +00:00
Author
Owner

Replying to davidsarah:

Well, I suppose we can only deprecate it after we've changed tahoe mv to stop using it, in any case.

I was mistaken, tahoe mv uses ?t=uri to add the link to the new location. It probably should be changed to use ?t=move, though.

Replying to [davidsarah](/tahoe-lafs/trac-2024-07-25/issues/1930#issuecomment-132640): > Well, I suppose we can only deprecate it after we've changed `tahoe mv` to stop using it, in any case. I was mistaken, `tahoe mv` uses `?t=uri` to add the link to the new location. It probably *should* be changed to use `?t=move`, though.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: tahoe-lafs/trac-2024-07-25#1930
No description provided.