[wp-trac] [WordPress Trac] #57979: Can't upload images to WordPress Comments

WordPress Trac noreply at wordpress.org
Fri Oct 13 20:21:04 UTC 2023


#57979: Can't upload images to WordPress Comments
-------------------------------------------------+-------------------------
 Reporter:  sbb                                  |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  6.4
Component:  Comments                             |     Version:  6.0.3
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch 2nd-opinion dev-feedback   |     Focuses:
  needs-testing                                  |  administration
-------------------------------------------------+-------------------------

Comment (by davidbaumwald):

 @nicolefurlan Thanks for the ping!  I actually don't think the latest
 patch is quite ready yet.  The unit tests are failing on the
 [https://github.com/WordPress/wordpress-develop/pull/4265 4265 PR] and I
 don't think it's the right approach.  Sergey's suggestion is more
 appropriate so that the commenter's capabilities are only checked if the
 current user's capes don't include `unfiltered_html`.

 I created [https://github.com/WordPress/wordpress-develop/pull/5485 PR
 #5485] to get that approach working, but we might need to rework the tests
 a bit.

 @SergeyBiryukov do you think that the expected result in
 `test_update_comment_from_unprivileged_user_by_privileged_user` should
 actually be reversed with this change?  If we allow a privileged user to
 post unfiltered HTML as proposed here, this test would need to be adjusted
 to check that `disallowed=attribute` is included in the expected result.
 Thoughts?

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/57979#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list