2

マジェント 1.4.1.1

歴史をさかのぼると、app/code/core/Mage/GoogleAnalytics/Block/Ga.phpapp/code/local/Mage/GoogleAnalytics/ Block/Ga.php にコピーされ、protected function _toHtml() {}関数が適切な Googleページ上の Analytics 非同期 JavaScript エントリ。オリジナルには、この Web サイトの追跡情報が Google に適切に送信されないという問題がありました。この書き直しは完全に機能し、望ましい結果が得られました。

物事を整理するために、この変更をローカル名前空間モジュールにモジュール化することが決定されました。以下のファイルが作成されました...

apps/etc/modules/Chief_GoogleAnalytics.xml

<?xml version="1.0"?>
<config>
    <modules>
        <Chief_GoogleAnalytics>
            <active>true</active>
            <codePool>local</codePool>
        </Chief_GoogleAnalytics>
    </modules>
</config>

アプリ/コード/ローカル/チーフ/GoogleAnalytics/etc/config.xml

<?xml version="1.0"?>
<config>
    <modules>
        <Chief_GoogleAnalytics>
            <version>0.1.0</version>
        </Chief_GoogleAnalytics>
    </modules>
    <global>
        <blocks>
            <googleanalytics>
                <rewrite>
                    <ga>Chief_GoogleAnalytics_Block_Ga</ga>
                </rewrite>
            </googleanalytics>
        </blocks>
    </global>
</config>

アプリ/コード/ローカル/チーフ/GoogleAnalytics/ブロック/Ga.php

/**
 * GoogleAnalitics Page Block
 *
 * @category   Chief
 * @package    Chief_GoogleAnalytics
 * @author     Magento Core Team <core@magentocommerce.com>
 */
class Chief_GoogleAnalytics_Block_Ga extends Mage_GoogleAnalytics_Block_Ga
{
    /**
     * Prepare and return block's html output
     *
     * @return string
     */
    protected function _toHtml()
    {
        if (!Mage::getStoreConfigFlag('google/analytics/active')) {
            return '';
        }

        $this->addText('
<!-- BEGIN GOOGLE ANALYTICS CODE -->
<script type="text/javascript">
//<![CDATA[
    var _gaq = _gaq || [];
    _gaq.push(["_setAccount", "' . $this->getAccount() . '"]);
    _gaq.push(["_trackPageview", "'.$this->getPageName().'"]);

    (function() {
        var ga = document.createElement(\'script\'); ga.type = \'text/javascript\'; ga.async = true;
        ga.src = (\'https:\' == document.location.protocol ? \'https://ssl\' : \'http://www\') + \'.google-analytics.com/ga.js\';
        var s = document.getElementsByTagName(\'script\')[0]; s.parentNode.insertBefore(ga, s);
    })();
//]]>
</script>
<!-- END GOOGLE ANALYTICS CODE -->
        ');

        $this->addText($this->getQuoteOrdersHtml());

        if ($this->getGoogleCheckout()) {
            $protocol = Mage::app()->getStore()->isCurrentlySecure() ? 'https' : 'http';
            $this->addText('<script src="'.$protocol.'://checkout.google.com/files/digital/ga_post.js" type="text/javascript"></script>');
        }

        return parent::_toHtml();
    }
}

app/code/local/Mage/GoogleAnalytics/Block/Ga.phpから来ていたときの通常のように修正された単一のスクリプト エントリの代わりに、重複が発生するようになりました。Chief_GoogleAnalytics ブロックの上書きが最初に行われ、その後にapp/code/core/Mage/GoogleAnalytics/Block/Ga.phpからの古いストック ジャンク コードが続きます。after_body_startに表示するように定義する Google アナリティクス用の単一のレイアウト XML ファイルがあるため、機能しています。これは、新しいブロックの上書きと古いブロックが表示されている複製です。何を与える?

<!-- BEGIN GOOGLE ANALYTICS CODE -->
<script type="text/javascript">
//<![CDATA[
    var _gaq = _gaq || [];
    _gaq.push(["_setAccount", "UA-xxxxxx-x"]);
    _gaq.push(["_trackPageview", "/"]);

    (function() {
        var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;
        ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';
        var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(ga, s);
    })();
//]]>
</script>
<!-- END GOOGLE ANALYTICS CODE -->

<!-- BEGIN GOOGLE ANALYTICS CODE -->
<script type="text/javascript">
//<![CDATA[
    (function() {
        var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;
        ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';
        (document.getElementsByTagName('head')[0] || document.getElementsByTagName('body')[0]).appendChild(ga);
    })();

    var _gaq = _gaq || [];
    _gaq.push(["_setAccount", "UA-xxxxxx-x"]);
    _gaq.push(["_trackPageview", "/"]);
//]]>
</script>
<!-- END GOOGLE ANALYTICS CODE -->

ブロックをロードする page.phtml エントリ。

<?php echo $this->getChildHtml('after_body_start') ?>

googleanalytics.xml レイアウト

<layout version="0.1.0">
    <default>
        <!-- Mage_GoogleAnalytics -->
        <reference name="after_body_start">
            <block type="googleanalytics/ga" name="google_analytics" as="google_analytics" />
        </reference>
    </default>
</layout>

戦略的な場所でMage::Log() withを 使用して追加されたフォレンジック ここでは配列オブジェクトを渡しているため、文字列を操作するよりも少し複雑です。$this->getData()

_toHtml() 関数の開始

2012-06-24T21:02:04+00:00 DEBUG (7): Array
(
    [type] => googleanalytics/ga
    [module_name] => Chief_GoogleAnalytics
)

すぐにロードさ parent::_toHtml(); せます $original_output =parent::_toHtml

2012-06-24T21:02:04+00:00 DEBUG (7): Array
(
    [type] => googleanalytics/ga
    [module_name] => Chief_GoogleAnalytics
    [account] => UA-xxxxxx-x
    [page_name] => /
    [text] => 
<!-- BEGIN GOOGLE ANALYTICS CODE -->
<script type="text/javascript">
//<![CDATA[
    (function() {
        var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;
        ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';
        (document.getElementsByTagName('head')[0] || document.getElementsByTagName('body')[0]).appendChild(ga);
    })();

    var _gaq = _gaq || [];
    _gaq.push(["_setAccount", "UA-xxxxxx-x"]);
    _gaq.push(["_trackPageview", "/"]);
//]]>
</script>
<!-- END GOOGLE ANALYTICS CODE -->

)

これが;tldrに変わるのを防ぐため$this->getData();に、戻りが最後にあった場所をログに記録すると、両方のjavascriptが吐き出されますが、元の逆になります。

$this->getData();参照せずparent::_toHtmlに、返されたオブジェクトに必要な次のものを渡します。適切な識別子を持つ 1 つのスクリプト...

2012-06-24T21:06:24+00:00 DEBUG (7): Array
(
    [type] => googleanalytics/ga
    [module_name] => Chief_GoogleAnalytics
    [account] => UA-xxxxxx-x
    [page_name] => /aircraft
    [text] => 
<!-- BEGIN GOOGLE ANALYTICS CODE -->
<script type="text/javascript">
//<![CDATA[
    var _gaq = _gaq || [];
    _gaq.push(["_setAccount", "UA-xxxxxx-x"]);
    _gaq.push(["_trackPageview", "/aircraft"]);

    (function() {
        var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;
        ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';
        var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(ga, s);
    })();
//]]>
</script>
<!-- END GOOGLE ANALYTICS CODE -->

)
4

2 に答える 2

6

まず、一見、しかし実際には衒学的ではない命名規則。書き換えではないファイルを配置するとlocal、コードプールのオーバーライドになります。カスタムモジュールを作成してrewriteノードを使用する場合、それはクラスの書き換えです。これらは2つの異なるアクションであり、それぞれの動作が異なります。これらの違いが、あなたが見ている行動を見ている理由です。

クラスオーバーライドを使用すると、

ねえMagento、のクラスファイルを含める代わりに、代わりMage_GoogleAnalytics_Block_Gaに私のものを含める

これにより、システム内の元のクラスが完全に置き換えられます。

クラスリライトを使用すると、

ねえMagento、()ブロックオブジェクトをインスタンス化するときは、代わりに私のクラスからオブジェクトをインスタンス化しますgoogleanalytics/gaMage_GoogleAnalytics_Block_Ga

次に、クラスを拡張することにより、元のMage_GoogleAnalytics_Block_Gaクラスに元のオブジェクトと同じように動作するオブジェクトがあり、メソッドを追加できます。ただし、メソッドを追加すると、その親メソッドは元のブロックになり、オーバーライドとは異なります。

ステップ1は、ローカルコードプールファイルを削除することです

app/code/local/Mage/GoogleAnalytics/Block/Ga.php

すでにこれを行っているようです。次に、書き直したクラスにメソッドがあります。parent::_toHtml();メソッドを呼び出すとき

protected function _toHtml()
{
    //... your stuff here ...
    return parent::_toHtml();
}

あなたはMagentoに「ねえ、元のオブジェクトがやったであろうことは何でもやる」と言っているのです。ブロックと_toHtmlメソッドを使用すると、同じ出力を生成することを意味します。そのため、「古いストックジャンクコード」を取得しています。メソッドのコントラクトは、_toHtml「これが返す文字列が何であれ、ブロック出力として含まれる」です。addText親メソッドが認識しているメソッドを使用しているため、新しいコードも含まれています。

したがって、_toHtmlブロックリライトでの動作を変更する場合は、次のような操作を行う必要があります。

protected function _toHtml()
{
    $original_output = parent::_toHtml();
    $string = '';
    //... your code here which ultimately generates your output
    //and places it in $string
    return $string;
}

最初に親メソッドを呼び出すことにより、状態の観点からブロックが実行することはすべて引き続き発生します(ブロックは実行すべきではありませんが、それがどのように行われるかはわかっています)。次に、代わりに文字列を返します。

于 2012-06-24T19:55:23.827 に答える
2

DDx:<block type="googleanalytics/ga" ... />からコードをコメントアウトすると、googleanalytics.xmlすべての出力が消えますか?

ローカルの codePool オーバーライド ファイルが削除されたと仮定すると、問題は確かにreturn parent::_toHtml(). 親 (コア) メソッドの CE 1.4.1.1 バージョンは、addText()から派生したものを使用しMage_Core_Block_Text->_toHtml()ます。addText()はコンテンツを に追加し_data['text']、メソッドはレンダリング中にこれを返します。したがって、オーバーライドは一部の文字列コンテンツを に設定し、親メソッドを呼び出すと、そのコンテンツを_data['text']に追加してからを呼び出します。これは単に の値を返します。_data['text']Mage_Core_Block_Text->_toHtml()_data['text']

一般的には、オーバーライドされたメソッドを呼び出すことをお勧めしますが、これは機能しない場合です。このメソッドが 1.4.2.0 でローカル変数から文字列を返すように変更されたことを考えると、親を無視しても問題ありません。奇妙な理由でこれがうまくいかない場合は、レイアウト XML を使用して、コア ブロック インスタンスを別のものに置き換えることができます。

于 2012-06-24T18:57:36.843 に答える