WordPress.org

Make WordPress Core

Opened 21 months ago

Last modified 12 months ago

#44111 new enhancement

Setting data to variable early.

Reported by: rnaby Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-patch needs-testing reporter-feedback close
Focuses: administration Cc:
PR Number:

Description

I think setting data to variable early.

Attachments (1)

44111.diff​ (1.1 KB) - added by rnaby 21 months ago.
The patch for the ticket.

Download all attachments as: .zip

Change History (4)

@rnaby
21 months ago

The patch for the ticket.

#1 @SergeyBiryukov
21 months ago

  • Keywords reporter-feedback close added; dev-feedback removed

Hi @rnaby, welcome to WordPress Trac!

The existing code looks fine to me as is. Is there a problem solved by this patch?

#2 @rnaby
21 months ago

@SergeyBiryukov Thanks for giving feedback. Actually, it's not solving problem, I proposed it for a cleaner approach. Like we are setting some data to a variable and if some condition has met only then we're changing it. Otherwise it's being what it was. I think it makes the code simple and a little less lengthy to read or understand.

#3 @pento
12 months ago

  • Version trunk deleted
Note: See TracTickets for help on using tickets.