[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