WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#15057 closed defect (bug) (wontfix)

Is the double serialization in maybe_serialize( ) intentional ?

Reported by: DrLightman Owned by:
Priority: normal Milestone:
Component: General Version: 3.0.1
Severity: normal Keywords: double, serialization
Cc:

Description 

function maybe_serialize( $data ) {
   if ( is_array( $data ) || is_object( $data ) )
       return serialize( $data );

   if ( is_serialized( $data ) )
       return serialize( $data );

   return $data;
}

   if ( is_serialized( $data ) )
       return serialize( $data );

It creeps me out :)

Change History (3)

comment:1 Denis-de-Bernardy — 3 years ago

What does SVN blame say on that line of code?

comment:3 scribu — 3 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

So, yeah, wontfix due to compatiblity reasons: [14074]

Note: See TracTickets for help on using tickets.