Vote #70283
完了Setting issue attributes from mail should be case-insensitive
0%
説明
Currently, when you want to set issue attribute, e.g. Status, from mail you would add (in the mail body):
Status: Closed
The attribute name can be set in either case: status, STATUS, etc., but it fails to recognize the attribute value unless it's set in title-case, i.e. exactly as it appears in web interface.
The attached patch fixes this problem by titleizeing certain attribute values extracted from the mail body.
journals
--------------------------------------------------------------------------------
Fixed in r6199.
This patch assumes that values are titleized in the database which may not be the case. The fix uses a different approach based on reusable scopes and applies to priorities and categories as well.
--------------------------------------------------------------------------------
Jean-Philippe Lang wrote:
> Fixed in r6199.
>
> This patch assumes that values are titleized in the database which may not be the case. The fix uses a different approach based on reusable scopes and applies to priorities and categories as well.
Thank you! I've had the feeling that my approach can be improved, but didn't know where to look exactly.
Btw, aren't we making code less DRY with all these @named_scope :named, ...@? Can be extracted in a module for sure.
--------------------------------------------------------------------------------
Alex Shulgin wrote:
> Btw, aren't we making code less DRY with all these @named_scope :named, ...@? Can be extracted in a module for sure.
Indeed.
--------------------------------------------------------------------------------