@Nemo_bis says in T848: Empty fields shouldn't be pasted as "none", "unspecified" or "unknown":
why the newline before last "none"?
This is a small problem now present in every task in bugzillapreview, i.e. https://bugzillapreview.wmflabs.org/T7238
@Nemo_bis says in T848: Empty fields shouldn't be pasted as "none", "unspecified" or "unknown":
why the newline before last "none"?
This is a small problem now present in every task in bugzillapreview, i.e. https://bugzillapreview.wmflabs.org/T7238
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Qgil | T553 Engineering Community team goals for October 2014 | |||
| Resolved | Qgil | T174 Launch Wikimedia Phabricator Day 1 | |||
| Resolved | Qgil | T15 Migrate Bugzilla to Phabricator | |||
| Resolved | • chasemp | T259 Bugzilla to Maniphest import script (tracking) | |||
| Resolved | • chasemp | T849 Newline after "See Also" field in bugzillapreview |
That's done because "See Also" can contain several URLs, see https://bugzillapreview.wmflabs.org/T1845 for an example: Currently one URL per line.
Code is at
https://git.wikimedia.org/blob/phabricator%2Ftools.git/7e65baec365bbfeb88219fa78d95c817073cd100/bugzilla_create.py#L186
Still valid in 2nd edition of bugzillapreview, e.g. seen on https://bugzillapreview.wmflabs.org/T69 (though I don't really care whether we fix this or not).
so the old case was
See Also: none
If none there is nothing present.
Now
See Also: http://urlofchoise.com
or
See Also: http//urlsabound.com http://moreurls.com
in the case of multiple url's not doing a newline is ugly, so I don't know if this issue as first reported is valid or not
So this looks indeed good:
See Also with one URL: https://bugzillapreview.wmflabs.org/T7
See Also with many URLs: https://bugzillapreview.wmflabs.org/T69