[wp-trac] [WordPress Trac] #50159: Simplepie 1.5.5 - code review and modifications

[WordPress Trac]

#50159: Simplepie 1.5.5 - code review and modifications
--------------------------------------+------------------------------
 Reporter:  arena                     |       Owner:  (none)
     Type:  defect (bug)              |      Status:  new
 Priority:  normal                    |   Milestone:  Awaiting Review
Component:  Feeds                     |     Version:  trunk
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+------------------------------

Comment (by afragen):

 Replying to [comment:14 arena]:
 > Yes, i have noticed that too. This is a replication of the samples
 provided by SimplePie.
 >
 > This transient has a 12 hours lifetime by default and if deprecated will
 be erased by appropriate wp-cron job 'delete_expired_transients'.
 >
 > I also changed some class names and added 2 new files
 >
 > Replying to [comment:12 afragen]:
 > > You have also changed the transient names.
 > >
 > >
 > > {{{
 > > 54                          $this->name     = 'feed_' . $filename;
 > > 55                          $this->mod_name = 'feed_mod_' . $filename;
 > > }}}
 > >
 > > to
 > >
 > >
 > > {{{
 > >
 > > 54                  $this->name     = 'feed_' . md5(
 "$filename:$extension" );
 > > 55                  $this->mod_name = 'feed_mod_' . md5(
 "$filename:$extension" );
 > >
 > > }}}
 > >
 > > This particular item will cause an issue for me.
 > > https://github.com/afragen/wordpress-beta-
 tester/blob/develop/src/WPBT/WPBT_Beta_RC.php#L520-L522


 I noticed the change, my point was why? There is potential incompatibility
 by changing and there is no need to change it.

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