1. This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

Implemented Allow XenForo_ProxyOutput to be extended

Discussion in 'Closed Suggestions' started by Jon W, Jan 28, 2014.

  1. Jon W

    Jon W Well-Known Member

    Please could you replace the static run method in XenForo_ProxyOutput in XenForo 1.3 with:
    PHP:
        /**
         * Static helper to execute a full request for proxy output
         */
        
    public static function run()
        {
            
    $dependencies = new XenForo_Dependencies_Public();
            
    $dependencies->preLoadData();

            
    $class XenForo_Application::resolveDynamicClass('XenForo_ProxyOutput');

            
    $proxyOutput = new $class($_REQUEST$dependencies);
            
    $proxyOutput->output();
        }
    I would really like to be able to extend this class.
     
    Kjartan Simonsen, goyo and arms like this.
  2. Mike

    Mike XenForo Developer Staff Member

    I have done this, but I'm not clear on why. Suggestions tend to be better if they explain motivations as well as the desired change (as it makes it more compelling).
     
    Jon W likes this.
  3. Jon W

    Jon W Well-Known Member

    Thanks, and point taken.

    In this case, I would like to add other proxies -- namely, a thumbnail proxy for my Thread Thumbnails add-on.
     
    yavuz likes this.

Share This Page