[wp-trac] [WordPress Trac] #18977: Make import.php dynamic

WordPress Trac noreply at wordpress.org
Thu Nov 15 20:12:23 UTC 2012


#18977: Make import.php dynamic
----------------------------+-----------------------
 Reporter:  jane            |       Owner:  nacin
     Type:  task (blessed)  |      Status:  accepted
 Priority:  normal          |   Milestone:  3.5
Component:  Import          |     Version:  3.3
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |
----------------------------+-----------------------

Comment (by dllh):

 I've added a new patch with the following modifications from the last:

 * Sets {{{$popular_importers}}} to false instead of unsetting it.
 * Checks for {{{is_array( $popular_importers)}}} rather than just testing
 for existence.

 I think this fixes the potential notice issue and is more
 consistent/robust; that is, if we're checking for an array in the one
 place, we should do the same below. This safeguards against the unlikely
 case that something else (a plugin, say) writes some non-array garbage
 into the transient.

 I suppose that if we wanted to be extra robust, we'd also check for
 {{{$popular_importers['importers']}}}, but maybe that's overkill.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/18977#comment:12>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list