Make WordPress Core

Opened 11 years ago

Closed 7 years ago

#35386 closed enhancement (invalid)

Refining Walker code

Reported by: Funkatronic Owned by:
Priority: normal Milestone:
Component: General Version:
Severity: normal Keywords: has-patch needs-unit-tests needs-refresh close
Cc: Focuses: performance

Description

This a simple patch for the Walker class. Previously, if root was not 0, a second for loop was performed in the walk function. This new code does the same functionality with less loops

Attachments (1)

walker_patch.patch​ (1.2 KB ) - added by Funkatronic 11 years ago.

Download all attachments as: .zip

Change History (6)

#1 @swissspidy
11 years ago

  • Keywords has-patch needs-unit-tests added

#2 @chriscct7
11 years ago

  • Version trunk

This ticket was mentioned in ​Slack in #core by noisysocks. ​View the logs.


7 years ago

#4 @talldanwp
7 years ago

  • Keywords needs-refresh added

Hi @Funkatronic. Thanks for creating this ticket and submitting the patch. I realise some time has passed, is this something you're still interested in following up on?

I tried applying the patch, but it looks like it was somehow generated with an absolute path (C:/Users/manny/AppData/Local/Temp/class-wp-walker.php-revBASE.svn000.tmp.php). Usually a patch would have a path relative to the root of the project like 'src/wp-includes/class-wp-walker.php'.

There are some guides to creating patches here if you're interested in refreshing this patch:

Thanks!

#5 @Funkatronic
7 years ago

  • Keywords close added
  • Resolution → invalid
  • Status new → closed
Note: See TracTickets for help on using tickets.