Big Brother TinyMCE for Revo

Since this package is more than 2MB, the downloading takes more time than the limit.
Apache’s error log:

PHP Fatal error: Maximum execution time of 30 seconds exceeded in D:\\localhost\\modxsite.com\\core\\model\\modx\\modsessionhandler.class.php on line 63, referer: http://localhost/modxsite.com/www/manager/index.php?a=61

I guess setting the time limit to 300 (Apache’s default) won’t harm the script.

And another error on MODx’s error page:

[2011-01-29 16:09:23] (ERROR @ D:\localhost\modxsite.com\core\xpdo\compression\xpdozip.class.php : 142) PHP warning: ZipArchive::extractTo(D:/localhost/modxsite.com/core/packages//tinymce-4.1.2-pl\modPlugin\57e0b943af880c9cee7bb55259f4f30f\0\tinymce\jscripts\autocomplete/jquery.autocomplete.min.js) [<a href=’ziparchive.extractto’>ziparchive.extractto</a>]: failed to open stream: Permission denied

I have a feeling that that double slash ’//’ makes the problem.

Adding this:

Makes TinyMCE downloaded successfully.

TinyMCE never took more then a few seconds to download for me...

Just tried locally and took three seconds.

Sounds to me like there’s a different problem going on causing it to take so long?

Maybe,
My server in Indonesia was waiting for the respond from MODx’s server in US.
That is almost a half way of the globe.

Quote from: goldsky at Jan 29, 2011, 04:39 AM

Adding this:

<?php


class modSessionHandler {
    /**
     * @var modX A reference to the modX instance controlling this session
     * handler.
     * @access public
     */
    public $modx= null;

    /**
     * Creates an instance of a modSessionHandler class.
     *
     * @param modX &$modx A reference to a {@link modX} instance.
     * @return modSessionHandler
     */
    function __construct(modX &$modx) {
        // Apache's timeout: 300 secs
        if (function_exists('ini_get') && !ini_get('safe_mode')) {
            if (function_exists('set_time_limit')) {
                set_time_limit(300);
            }
            if (function_exists('ini_set')) {
                if (ini_get('max_execution_time') !== 300) {
                    ini_set('max_execution_time', 300);
                }
            }
        }
        
        $this->modx= & $modx;
    }

Makes TinyMCE downloaded successfully.

hei goldsky, where did you add this?

/core/model/modx/modsessionhandler.class.php

You shouldn’t be hacking the core code like that. You just need to either:

a) Up your max_execution_time setting in your php.ini
b) Change the MODX System Setting archive_with to 1 to use a different zip archiving method.

Quote from: splittingred at Jan 31, 2011, 07:47 AM

You shouldn’t be hacking the core code like that. You just need to either:

a) Up your max_execution_time setting in your php.ini
b) Change the MODX System Setting archive_with to 1 to use a different zip archiving method.

I’m afraid that this way does not work either.
I tried by putting the php.ini in the manager/ folder.
Setting archive_with to 1 made thing even worse:

[2011-01-31 22:57:12] (ERROR @ /modxsite.com/www/connector/workspace/packages-rest.php) Could not open file for reading: http://modxcms.com/assets/components/releaseme/dl.php?file=1847&revolution_version=Revolution-2.0.7-pl
[2011-01-31 22:57:12] (ERROR @ /modxsite.com/www/connector/workspace/packages-rest.php) MODx could not download the file. You must enable allow_url_fopen, cURL or fsockopen to use remote transport packaging.
[2011-01-31 22:57:12] (ERROR @ /modxsite.com/www/connector/workspace/packages-rest.php) Could not download package at: http://modxcms.com/assets/components/releaseme/dl.php?file=1847

Sad but true, my hack did the trick.
I just tested it again.

Hmm...
If I could make a provider in my server here, you could test it.