We launched new forums in March 2019—join us there. In a hurry for help with your website? Get Help Now!
    • 33982 ☆ A M B ☆
    • 476 Posts
    Is there a way to turn off the warning altogether? I’m not sure exactly which document it’s in, but if anyone does know, I’d love to comment out the code that puts out that warning. The warning is not an issue that affects the sites I have, so it turns into one of those things that bugs my clients which in turn means they’re going to bug me about it sad Otherwise, thnx for the upgrade! smiley
      Precision Web Development ... SmashStack.com
      • 4041
      • 788 Posts
      Try commenting out lines 24,25,26,27 of manager/includes/config_check.inc.php and see if that does it.
        xforum
        http://frsbuilders.net (under construction) forum for evolution
        • 33982 ☆ A M B ☆
        • 476 Posts
        Try commenting out lines 24,25,26,27 of manager/includes/config_check.inc.php and see if that does it.

        I tried that and it produced an error so I then removed the comments and then reuploaded the config_check.inc.php and the Configuration warning message disappeared which was my initial goal. I’m not entirely sure why that happened, but the warning message is gone now even though I uncommented those lines...weird.
          Precision Web Development ... SmashStack.com
          • 5913
          • 42 Posts
          Hey All,

          Ok you only need to comment out lines 25 and 26 in the manager/includes/config_check.inc.php as that suppresses the warning.
          (if you are unable to add the php zip to your php.ini)

          Always remember to add a note somewhere indicating why you modified it so that you know when you go to do the next upgrade what and why you did smiley
            • 28042 ☆ A M B ☆
            • 24,524 Posts
            I’m sure the next upgrade will be able to deal with this in some way; have a "don’t show this again" checkbox on the error page maybe? Only show it to #1 admin user?
              Studying MODX in the desert - http://sottwell.com
              Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
              Join the Slack Community - http://modx.org
              • 33982 ☆ A M B ☆
              • 476 Posts
              I’m sure the next upgrade will be able to deal with this in some way; have a "don’t show this again" checkbox on the error page maybe? Only show it to #1 admin user?

              That’d be a nice function for sure.
                Precision Web Development ... SmashStack.com
                • 28042 ☆ A M B ☆
                • 24,524 Posts
                For now, I have a very simple plugin; it uses the OnManagerWelcomeHome event and contains one line:
                $modx->placeholders['config_display'] = 'none';

                And *poof* no more red Configuration tab!

                You could fiddle with this and make it only if the user is not #1 Admin user, or anything else you wanted.
                  Studying MODX in the desert - http://sottwell.com
                  Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
                  Join the Slack Community - http://modx.org
                  • 28042 ☆ A M B ☆
                  • 24,524 Posts
                  Ok, here’s a more selective plugin that just removes the GD and Zip error. It’s really not a good idea to be removing all errors; one of them might be important. Remove the leading <?php tag if you use this.
                  <?php
                  // turn it into an array for easier handling, 
                  //there are too many HTML tags and line endings to work directly with the error string
                  $errors = explode('Configuration warning', $modx->placeholders['config_check_results']);
                  // dig through the array for the GD and Zip warning
                  foreach($errors as $key=>$value) {
                      if(strpos($value, 'GD and Zip') !==false) $gdkey = $key;
                  }
                  //drop the GD and Zip warning
                  unset($errors[$gdkey]);
                  // the first element will be the error list heading
                  if(count($errors)>1) {
                          // put it back together
                          $string = implode('Configuration warning', $errors);
                          // replace the error message
                          $modx->placeholders['config_check_results'] = $string;
                  } else {
                          // no other errors, don't show the tab at all
                          $modx->placeholders['config_display'] = 'none';
                  }
                  return;
                  
                    Studying MODX in the desert - http://sottwell.com
                    Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
                    Join the Slack Community - http://modx.org
                    • 27421
                    • 67 Posts
                    Works great! Thanks.
                      • 37550 ☆ A M B ☆
                      • 711 Posts
                      Hi,

                      For any Onesmarthost customers we will be updating our servers PHP Configuration late this evening to enable PHP Zip support which should remove this warning.

                      Aaron


                        http://www.onesmarthost.co.uk
                        UK MODX Hosting with love.