Jump to content
Sign in to follow this  
k00ni

[pSys: Installation] Schritt 2 - Korrigierte Darstellung des SafeMode Status

Recommended Posts

pSys Version: 0.7.3 alpha


Fehlerbeschreibung:


In der install/index.php ca. bei Zeile 126 folgendes:

 

// Safe-Mode prüfen: 0 = aus, 1 = an.
if ( ini_get('safe_mode') != "off" ) 
{
      $probcheck = 1;
      echo "";
} 
else 
{
       echo "";
}

echo " Safe Mode ".ini_get('safe_mode') ." - benötigt wird: off";

 

durch dieses ersetzen:

 

// Safe-Mode prüfen: 0 = aus, 1 = an.
if ( ini_get('safe_mode') != 0 ) 
{
      $probcheck = 1;
      echo "";
$S_safe_mode_status = 'on';
} 
else 
{
       echo "";
$S_safe_mode_status = 'off';
}

echo " Safe Mode ". $S_safe_mode_status ." - benötigt wird: off";

 

Damit bekommt man einen korrigierten Safe-Mode-Status angezeigt. Die Funktion ini_get('safe_mode') gibt nämlich nur 0 oder 1 zurück und es wird hier aber mit "off" verglichen, weiterhin unten auch einfach ausgegeben, daneben steht aber


benötigt wird: off


was verwirren könnte. Nun also eine einheitliche Ausgabe. :)



Grüße

Share this post


Link to post
Share on other sites

Sehe gerade dass die Version in der News von der hier (Installation, Schritt 2) geforderten abweicht. Würde ich vielleicht noch ändern:

 

//php Version

if ( version_compare( $phpver, "5.1" ) ";

} else {

      echo "";

}

echo " php Version $phpver - benötigt: 5.1.x";

 

ändern in

 

//php Version

                   if ( version_compare( $phpver, "5.1" ) ";

                   } else {

                       echo "";

                   }

                   echo " php Version $phpver - benötigt: 5.1.x";

Share this post


Link to post
Share on other sites

Mal sehen, ich habe ein gänzlich anderes Installationsscript bereits in der Mache, ich werde mich am alten nicht mehr weiter betätigen :wink:

Share this post


Link to post
Share on other sites
Guest
You are commenting as a guest. If you have an account, please sign in.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this  

×