Korrektur in OrderHistoryBoxThemeContentView.inc.php updatesicher?

Thema wurde von jann_brand, 24. Januar 2020 erstellt.

  1. jann_brand

    jann_brand Aktives Mitglied

    Registriert seit:
    17. Januar 2020
    Beiträge:
    35
    Danke erhalten:
    0
    Hallo

    Ich habe bei mir die funktion add_product_data() in der OrderHistoryBoxThemeContentView.inc.php angepasst, da sonst die Links nicht generiert wurden. Es hiess dort erst

    Code:
    if ($this->coo_seo_boost->boost_products) {
                $product_link = xtc_href_link($this->coo_seo_boost->get_boosted_product_url($row->products_id,
                    $row->products_name));
            } else {
                $product_link = xtc_href_link(FILENAME_PRODUCT_INFO,
                    xtc_product_link($row->products_id, $row->products_name));
            }

    Wobei >result_array() ein array zurückliefert, geht dann natürlich nicht die Felder mit dem Objektsyntax anzusprechen. Ist die Änderung updatesicher? Wie muss ich vorgehen, dass sie das wird?

    Danke und lg

    Code:
    protected function add_product_data()
    {
        $result = $this->db->reset_query()
            ->select("products_id, products_name, products_meta_description")
            ->from(TABLE_PRODUCTS_DESCRIPTION)
            ->where_in("products_id", $this->product_ids_array)
            ->where("language_id = '{$this->language_id }'")
            ->order_by("products_name")
            ->get()
            ->result_array();
    
        foreach ($result as $row) {
            if ($this->coo_seo_boost->boost_products) {
                $product_link = xtc_href_link($this->coo_seo_boost->get_boosted_product_url($row['products_id'],
                    $row['products_name']));
            } else {
                $product_link = xtc_href_link(FILENAME_PRODUCT_INFO,
                    xtc_product_link($row['products_id'], $row['products_name']));
            }
            $t_title = '';
    
            if ($row['products_meta_description'] !== '') {
                if (strlen_wrapper($row['products_meta_description']) > 80) {
                    $t_title = htmlspecialchars_wrapper(substr_wrapper($row['products_meta_description'], 0, 80));
                } else {
                    $t_title = htmlspecialchars_wrapper($row['products_meta_description']);
                }
            }
    
            $this->content_array['orderHistoryProducts'][] = [
                'url' => $product_link,
                'title' => $t_title,
                'text' => $this->truncate($row['products_name'], gm_get_conf('TRUNCATE_PRODUCTS_HISTORY')),
            ];
        }
    }
     
  2. Wilken (Gambio)

    Wilken (Gambio) Administrator
    Mitarbeiter

    Registriert seit:
    7. November 2012
    Beiträge:
    17.851
    Danke erhalten:
    6.800
    Danke vergeben:
    2.060
  3. jann_brand

    jann_brand Aktives Mitglied

    Registriert seit:
    17. Januar 2020
    Beiträge:
    35
    Danke erhalten:
    0
    Ich habe es nicht versucht, sondern ich habe es gemacht. Wir werden die Änderung dann einfach beibehalten und bei einem eventuell neuen Upgrade würde da korrigiert.