csvimport: convert all instances of "SELECT *"
authorDenis Ovsienko <denis@ovsienko.info>
Sun, 7 Jun 2020 12:28:45 +0000 (13:28 +0100)
committerDenis Ovsienko <denis@ovsienko.info>
Sun, 7 Jun 2020 14:02:43 +0000 (15:02 +0100)
csvimport/plugin.php

index 5fe3523e469dec2d0ca30bbe5d64a20040382481..01d37ec271ca0e35e024e6d54f0e832958d25ba8 100644 (file)
@@ -678,7 +678,7 @@ function addCableLink($csvdata,$row_number)
        $cable_id =     trim ($csvdata[5]);
 
        // Check if object_a and port_a exist, if not; stop and return false
        $cable_id =     trim ($csvdata[5]);
 
        // Check if object_a and port_a exist, if not; stop and return false
-       $result = usePreparedSelectBlade ("SELECT Port.*, Object.name FROM Port, Object WHERE Port.object_id = Object.id AND Port.name='".$port_a."' AND Object.name='".$object_a."';");
+       $result = usePreparedSelectBlade ("SELECT Port.id, Object.name FROM Port, Object WHERE Port.object_id = Object.id AND Port.name='".$port_a."' AND Object.name='".$object_a."';");
        $db_result_a = $result->fetch (PDO::FETCH_ASSOC);
        if (!$db_result_a)
        {
        $db_result_a = $result->fetch (PDO::FETCH_ASSOC);
        if (!$db_result_a)
        {
@@ -687,7 +687,7 @@ function addCableLink($csvdata,$row_number)
        }
 
        // Check if object_a and port_a exist, if not; stop and return false
        }
 
        // Check if object_a and port_a exist, if not; stop and return false
-       $result = usePreparedSelectBlade ("SELECT Port.*, Object.name FROM Port, Object WHERE Port.object_id = Object.id AND Port.name='".$port_b."' AND Object.name='".$object_b."';");
+       $result = usePreparedSelectBlade ("SELECT Port.id, Object.name FROM Port, Object WHERE Port.object_id = Object.id AND Port.name='".$port_b."' AND Object.name='".$object_b."';");
        $db_result_b = $result->fetch (PDO::FETCH_ASSOC);
        if (!$db_result_b)
        {
        $db_result_b = $result->fetch (PDO::FETCH_ASSOC);
        if (!$db_result_b)
        {
@@ -808,7 +808,7 @@ function addIP($csvdata,$row_number)
        // Check if vlan domain - vlan combination exists
        if ((strlen($vlan_domain) > 0) & (strlen($vlan_id) > 0))
        {
        // Check if vlan domain - vlan combination exists
        if ((strlen($vlan_domain) > 0) & (strlen($vlan_id) > 0))
        {
-               $result = usePreparedSelectBlade ("SELECT VLANDescription.*, VLANDomain.description FROM VLANDescription, VLANDomain WHERE VLANDomain.id = VLANDescription.domain_id AND VLANDescription.vlan_id=".$vlan_id." AND VLANDomain.description ='".$vlan_domain."';");
+               $result = usePreparedSelectBlade ("SELECT VLANDescription.domain_id, VLANDomain.description FROM VLANDescription, VLANDomain WHERE VLANDomain.id = VLANDescription.domain_id AND VLANDescription.vlan_id=".$vlan_id." AND VLANDomain.description ='".$vlan_domain."';");
                $vlan_result = $result->fetch (PDO::FETCH_ASSOC);
                if (!$vlan_result)
                {
                $vlan_result = $result->fetch (PDO::FETCH_ASSOC);
                if (!$vlan_result)
                {
@@ -878,7 +878,7 @@ function setObjectAttributes($csvdata,$row_number)
 
        if (strlen($object ) > 0)
        {
 
        if (strlen($object ) > 0)
        {
-               $result = usePreparedSelectBlade ("SELECT  Object.* FROM Object WHERE Object.name='".$object."';");
+               $result = usePreparedSelectBlade ("SELECT id, name, label, asset_no, has_problems, comment FROM Object WHERE Object.name='".$object."';");
                $db_object = $result->fetch (PDO::FETCH_ASSOC);
 
                // Go ahead when object exists
                $db_object = $result->fetch (PDO::FETCH_ASSOC);
 
                // Go ahead when object exists