[buddypress-trac] [BuddyPress Trac] #7324: Possible php warnings in bp_legacy_theme_delete_activity_comment()

buddypress-trac noreply at wordpress.org
Mon Dec 5 22:41:20 UTC 2016


#7324: Possible php warnings in bp_legacy_theme_delete_activity_comment()
---------------------------+----------------------
 Reporter:  Jonas Lundman  |       Owner:  slaFFik
     Type:  defect (bug)   |      Status:  closed
 Priority:  lowest         |   Milestone:  2.8
Component:  Core           |     Version:
 Severity:  normal         |  Resolution:  fixed
 Keywords:  has-patch      |
---------------------------+----------------------
Changes (by slaffik):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"11270"]:
 {{{
 #!CommitTicketReference repository="" revision="11270"
 Prevent possible warning in `bp_legacy_theme_delete_activity_comment()`.

 Firstly, check `$_POST['id']` itself, only after that - for logged-in user
 status. Prevents warnings when the AJAX request is sent with wrong params,
 which doesn't allow to proceed and run additional code.

 Props jonas-lundman.
 Fixes #7324.
 }}}

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7324#comment:7>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list