[wp-trac] [WordPress Trac] #47050: Recovery Email - Keep consistent newline spacing around ###CAUSE### mergetag
WordPress Trac
noreply at wordpress.org
Fri Apr 26 05:29:22 UTC 2019
#47050: Recovery Email - Keep consistent newline spacing around ###CAUSE###
mergetag
-----------------------------+------------------------------
Reporter: garrett-eclipse | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Administration | Version: trunk
Severity: minor | Resolution:
Keywords: has-patch | Focuses:
-----------------------------+------------------------------
Changes (by garrett-eclipse):
* severity: normal => minor
Comment:
Replying to [comment:2 pento]:
> This is intentional, as the cause can also be empty, so we don't want
masses of whitespace in the email if there's no cause listed.
>
> I'm fine with reworking the email text to make it consistent, but that
needs to be taken into account.
Thanks @pento I didn't realize there was the case of an empty cause when
$extension isn't set (I've moved this down to minor). I do wonder if a
'Unknown Cause' string could be more useful than no cause. Or possibly
reworking the cause section in a future iteration to provide a 'Cause: '
prefix on it's own line and either supply the cause or an 'N/A' (or
Unknown).
*If this type of prefix is used then the existing cause strings won't need
the verbose 'In this case, ' prefix.
Cheers
--
Ticket URL: <https://core.trac.wordpress.org/ticket/47050#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list