Jump to content

[Possible bug in IPS] CMS module throws notice


VarunAgw

Recommended Posts

\applications\cms\sources\Databases\Databases.php:hasReciprocalLinking($databaseId)

if ( $data['database'] == $databaseId and $data['crosslink'] )

This line in hasReciprocalLinking() method throws notice error for me.

 

  14 => 
    array (size=7)
      0 => 
        array (size=1)
          'database' => string '10' (length=2)
      1 => 
        array (size=1)
          'database' => string '12' (length=2)
      2 => 
        array (size=1)
          'database' => string '12' (length=2)
      3 => 
        array (size=1)
          'database' => string '7' (length=1)
      4 => 
        array (size=1)
          'database' => string '26' (length=2)
      5 => 
        array (size=1)
          'database' => string '13' (length=2)
      6 => 
        array (size=1)
          'database' => string '13' (length=2)
  15 => 
    array (size=1)
      0 => 
        array (size=1)
          'database' => string '13' (length=2)
  16 => 
    array (size=4)
      0 => 
        array (size=1)
          'database' => string '7' (length=1)
      1 => 
        array (size=2)
          'database' => string '16' (length=2)
          'crosslink' => boolean true
      2 => 
        array (size=2)
          'database' => string '16' (length=2)
          'crosslink' => boolean true
      3 => 
        array (size=1)
          'database' => string '9' (length=1)
  17 => 
    array (size=3)
      0 => 
        array (size=1)
          'database' => string '16' (length=2)

That's how the $values object looks like. Note when crosslink is false, it's not set to false by IPS in $values, instead the crosslink key is removed from the array, which causes `if ( $data['database'] == $databaseId and $data['crosslink'] )` to throw a notice.

 

Edited by VarunAgw
Link to comment
Share on other sites

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...