[buddypress-trac] [BuddyPress Trac] #4187: Date field on registration
buddypress-trac
noreply at wordpress.org
Thu Dec 8 02:43:27 UTC 2016
#4187: Date field on registration
----------------------------------------+-----------------------------
Reporter: Mqlte | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Future Release
Component: Members | Version: 1.5.5
Severity: minor | Resolution:
Keywords: dev-feedback needs-refresh |
----------------------------------------+-----------------------------
Changes (by boonebgorges):
* keywords: has-patch dev-feedback => dev-feedback needs-refresh
Comment:
@lakrisgubben This is really great - thanks for the patch. I'm having a
hard time applying it, though - I get a "patch unexpectedly ends in middle
of line" error. Can you try regenerating it? (If it's very hard to do, I
can try applying it manually.)
The whole idea of modifying `$_POST` values like this is a bit unsettling
to me. I wonder if we might take this opportunity to put the values of
`$_POST` into a separate variable before modifying. (This, in turn, would
make it easier to write unit tests.)
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4187#comment:9>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list