Make WordPress Core

Opened 16 months ago

Closed 16 months ago

Last modified 16 months ago

#61814 closed defect (bug) (fixed)

Clarify the description for max_file_uploads in Site Health Info

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.7 Priority: normal
Severity: normal Version: 5.5
Component: Site Health Keywords: good-first-bug has-patch
Focuses: Cc:

Description

Background: #50038

The current desciption for max_file_uploads on Site Health Info screen is:

Max number of files allowed

That seems confusing to me. Allowed where exactly?

The description from the PHP Manual is clearer:

The maximum number of files allowed to be uploaded simultaneously

However, that's a bit long as a label, and wraps to two lines in the table.

I would suggest this instead:

Max simultaneous file uploads

Attachments (1)

61814.patch (1.2 KB) - added by iflairwebtechnologies 16 months ago.

Download all attachments as: .zip

Change History (10)

#1 @iflairwebtechnologies
16 months ago

@SergeyBiryukov
You can review the created 61814.patch and uploaded

This ticket was mentioned in PR #7142 on WordPress/wordpress-develop by @hbhalodia.


16 months ago
#2

  • Keywords has-patch added; needs-patch removed

#3 @hbhalodia
16 months ago

Hi @SergeyBiryukov @iflairwebtechnologies, Apologies for the PR raised, I did check the ticket after I raised the PR and found that the patch was already added a few min back. Though I guess a PR is needed on the wordpress-develop repository to keep the track.

Last edited 16 months ago by hbhalodia (previous) (diff)

@mukesh27 commented on PR #7142:


16 months ago
#5

We already have #7144. closing this.

#6 @mukesh27
16 months ago

  • Keywords changes-requested added

@iflairwebtechnologies Left one feedback.

#7 @SergeyBiryukov
16 months ago

  • Keywords changes-requested removed

#8 @SergeyBiryukov
16 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 58857:

Site Health: Clarify the description for max_file_uploads in Site Health Info.

Follow-up to [48535].

Props iflairwebtechnologies, hbhalodia, mukesh27, SergeyBiryukov.
Fixes #61814.

@SergeyBiryukov commented on PR #7144:


16 months ago
#9

Thanks for the PR! Merged in r58857.

Note: See TracTickets for help on using tickets.