1078009 - Changes/history now include some wrong field names

Status: RESOLVED WONTFIX

(bugzilla.mozilla.org :: Extensions, defect)

Description

[Kohei Yoshino [:kohei]]

Found a regression on my BzDeck app. For example,

https://bugzilla.mozilla.org/rest/bug/935719/history

Here, field names like "flagtypes.name" and "attachments.isobsolete" are wrong or backward incompatible. These should be "flag" and "attachment.is_obsolete", as per the config:

https://bugzilla.mozilla.org/config.cgi?ctype=json

Comment 1

[Kohei Yoshino [:kohei]]

Or maybe field names in config.cgi are outdated? At least "flagtypes.name" and "attachments.*" are incompatible. The new field names matches ones in https://bugzilla.mozilla.org/rest/field/bug

I have added a workaround to my app, but it would be nice if the mismatches get fixed in Bugzilla.

https://github.com/kyoshino/bzdeck/commit/74e40a1bd72b13d8dc4a1d99cb59ce4207d5cfc7

Comment 2

[Frédéric Buclin]

dkl: is this bug valid? If yes, is it a blocker for 5.0?

Comment 3

[David Lawrence [:dkl]]

(In reply to Frédéric Buclin from comment #2)
> dkl: is this bug valid? If yes, is it a blocker for 5.0?

Not a blocker IMO. Sorry for not addressing this bug til now. https://bugzilla.mozilla.org/config.cgi?ctype=json is part of the BzAPI extension is not part of the native REST API so there will be differences. There is a bug to do a proper /rest/configuration (bug 504937) which should address that missing config part of the native API. /rest/bug/<bug>/history has always returned the internal field name for 'flags' as it is not part of the Bugzilla::Bug::FIELD_MAP. We can address that as part of this bug.

dkl

Comment 4

[Kohei Yoshino [:kohei]]

Ah, the original report is about the BzAPI extension's configuration method, not config.cgi?ctype=json, but anyway if /rest/configuration (Bug 504937) does the right thing, that'd be great.