Door
haubebla haubebla
op 27-01-2014 19:12
gewijzigd op 27-01-2014 19:12
2.054 views
Hoi PHPhulp!
Ik kom een heel idioot probleem tegen:
Ik heb een webpagina class, die voor mij modules kan laden.
Als afgeleiden daarvan voor een van mijn websites wordt in de __constructor de module geladen MOD_TIME, die eigenlijk dit include't:
return $aDateParts[2].' '.$aDateParts[1].' '.$aDateParts[0];
}
//dit is test
echo 'YOYOYOGHURT';
echo ' ',(int)function_exists('parse_date');
?>
[/code]
Vervolgens gebruik ik in een include van de 'echte' pagina de functie parse_date().
Dit resulteert in een error:
Fatal error: Call to undefined function parse_date() in C:\Users\Ravi\xampp\htdocs\xampp\www\Ravithrasworld.nl\CLEAN Ravithrasworld.nl\SIS\kalender
Raar, want boven aan de pagina staat toch echt "YOYOYOGHURT 1".
De functie is dus eerst wel gedefineerd, maar dan ineens niet.
Zefs als ik dit doe:
[code file=Include]
<div id="kalender">
<h3 style="color: red;">
<?php
include MOD_TIME;
echo parse_date(date('Y-m-d')); #Lijn 5
?>
</h3>
...
</div>
[/code]
Geeft hij eerst "YOYOYOGHURT 1", maar daarna bovenstaande foutmelding.
Kunnen jullie me helpen met dit idiote probleem?
"Als afgeleiden daarvan voor een van mijn websites wordt in de __constructor de module geladen MOD_TIME"
Je bedoeld dat je een include statement in je __construct methode hebt staan?
Klinkt als een vreemde plaats om een dependency te includen. Normaal doe je dat voor de class definitie.
Er staat geen include in de constructor, maar $this->load_module(MOD_TIME).
Zie hier mijn klasse:
<?php
/**
* Constante MOD_SWIFT
* @see webclass
* @package modules
* Module voor swift mailer
*/
define('MOD_SWIFT','http://cache.ravithrasworld.nl/swift_require.php'); #TODO
/**
* Constante MOD_MYSQLII
* @see webclass
* @package modules
* Module voor object-georiënteerde bibliotheek voor databases, Mysql Improved Improved
*/
define('MOD_MYSQLII','http://cache.ravithrasworld.nl/mysqliI.class.php'); #TODO
/**
* Constante MOD_CLEAN_SOURCE
* @see webclass
* @package modules
* Module voor een schone broncode
*/
define('MOD_CLEAN_SOURCE','http://cache.ravithrasworld.nl/clean_source.php'); #TODO
/**
* Constante MOD_BOOLV2
* @see webclass
* @package modules
* Module voor extra constanten voor return's van functies
*/
define('MOD_BOOLV2','http://cache.ravithrasworld.nl/boolv2.php');
/**
* Constante MOD_FORMS
* @see webclass
* @package modules
* Module voor object-georiënteerde bibliotheek voor formulieren in HTML5
*/
define('MOD_FORMS','http://cache.ravithrasworld.nl/forms.class.php');
/**
* Constante MOD_TIME
* @see webclass
* @package modules
* Module voor procedurele bibliotheek met functies voor tijd.
*/
define('MOD_TIME','http://cache.ravithrasworld.nl/time.php');
/**
* Main::show($var)
* @param $var string, de variabele die geprint moet worden
* @param $exit bool, bepaald of het script moet worden afgebroken (standaard true)
* @return output
* @category View
*/
function show($var,$exit=true) {
echo '<pre>';
print_r($var);
echo '<pre>';
if ($exit) {
exit;
}
}
class html5 {
/**
* html5::$sTitel
* @var string(64)
* @access Om te wijzigen, gebruik html5::__contruct()
* De titel van de webpagina, wordt ingesteld met html5::__construct()
*/
protected $sTitel = NULL;
/**
* html5::$aStyleSheets
* @var array
* @access Om te wijzigen, gebruik html5::define_css() om te wijzigen en gebruik html5::defined_css() om te bekijken
* @package css
* De stylesheets om te laden in de webpagina, kan alleen gewijzigd worden voordat html5::toon_head() is gebruikt
*/
protected $aStyleSheets = array();
/**
* html5::$sInlineCSS
* @var string(long)
* @access Om te wijzigen, gebruik html5::define_inline_css() om te wijzigen.
* @see html5::$aStyleSheets
* @package css
* De inline-style tussen de style-tags, kan alleen gewijzigd worden voordat html5::toon_head() is gebruikt
*/
protected $sInlineCSS = NULL;
/**
* html5::$sJavaScript
* @var string(long)
* @access Om te wijzigen, gebruik html5::define_javascript om te wijzigen.
* @package javascript
* Het script tussen de 'script' tags.
*/
protected $sJavaScript = NULL;
/***** INTERNAL PROPERTIES *****/
/**
* html5::$bHeadView
* @var bool, standaard false
* @access Dit is een interne variabele, geen toegang mogelijk!
* @internal
* @see html5::toon_head()
* Of er een header met html5::toon_head() is getoond
*/
protected $bHeadView = false; #INTERN
/**
* html5::$aIncludeBestanden
* @var array
* @access Gebruik html5::include_add() om een index toe te voegen
* @internal
* @package includes
* Een array met de paden van de te includeren bestanden.
*/
protected $aIncludeBestanden = array(); #INTERN
/**
* html5::$bJQueryLoad
* @var bool, standaard false
* @access Gebruik html5::load_JQuery() om JQuery te laden.
* @access Gebruikt de tweede parameter van html5::define_javascript()
* @internal
* @package javascript
* @see html5::load_JQuery()
*/
protected $bJQueryLoad = false; #INTERN
/***** METHODES *****/
/**
* html5::__construct($spTitel)
* @category Model
* @param $spTitel string(64), de titel van een webpagina.
* @return void
* Verstuurd standaard HTTP-headers en schakelt compressie in.
* Titel wordt ingesteld.
*/
function __construct($spTitel) {
header('Content-Language: nl-NL');
// HTTP-header voor HTML plus de karakterset
header('Content-Type: text/html; charset=UTF-8');
ob_start('ob_gzhandler');
if (strlen($spTitel) <= 64) {
$this->sTitel = $spTitel;
} else {
trigger_error('Titel is langer dan 64 tekens',E_USER_NOTICE);
$this->sTitel = $spTitel;
}
}
/**
* html5::__destruct()
* @category automatisch
* Sluit de webpagina af.
*/
function __destruct() {
if (!$this->bHeadView) {
trigger_error('Er is geen header getoond',E_USER_NOTICE);
}
echo "\r\n\r\n</body>\r\n</html>";
$this->flush();
}
/**
* html5::toon_head()
* @category View
* Laat de header zien.
* Na deze functie kan er geen CSS of JavaScript meer geladen worden.
*/
function toon_head() {
$this->bHeadView = true;
echo "<!DOCTYPE html>\r\n\r\n";
echo "<html lang=\"nl-NL\">\r\n";
echo "<head>\r\n";
echo "<meta http-equiv=\"Content-Language\" content=\"nl-NL\">\r\n";
echo "<meta http-equiv=\"Content-Type\" content=\"text/html; charset=UTF-8\">\r\n";
echo "<meta charset=\"UTF-8\">\r\n";
echo "<meta name=\"robots\" content=\"index,follow\">\r\n";
foreach($this->aStyleSheets as $key=>$value) {
echo "<link rel=\"stylesheet\" href=\"$value\">\r\n";
}
echo "<title>" , $this->sTitel , "</title>\r\n";
if (!empty($this->sInlineCSS)) {
echo '<style type="text/css">',"\r\n";
echo $this->sInlineCSS;
echo "\r\n</style>\r\n\r\n";
}
if ($this->bJQueryLoad) {
echo "<script src=\"http://ajax.googleapis.com/ajax/libs/jquery/1.10.2/jquery.min.js\"></script>\r\n";
}
if (!empty($this->sJavaScript)) {
echo "<script>",$this->sJavaScript,"</script>\r\n";
}
echo "</head>\r\n\r\n";
echo "<body lang=\"nl-NL\">\r\n";
$this->flush();
}
/**
* html5::flush()
* @category View
* Flush't de output-buffers met main::flush() en ob_gzhandler::ob_flush()
*/
function flush() {
ob_flush();
flush();
}
/**
* html5::redirect_to($page)
* @category Controller
* @param $page path, het pad van het naar te herleiden bestand.
* @return EXIT
* Verstuurd een redirect-header en breekt het script af.
*/
function redirect_to($page) {
if (file_exists($page) && !headers_sent()) {
header('location: '.$page);
exit;
} elseif (headers_sent()) {
trigger_error('Er zijn al headers verzonden',E_USER_WARNING);
} else {
trigger_error('Bestand '.$page.' niet gevonden',E_USER_WARNING);
}
}
/**
* html5::load_modules(infinity)
* @category Model
* @package Modules
* @param infinity, constantes met de prefix MOD_
* @return false bij faling, main::require_once() bij succes
*
* Laad de opgegeven modules.
* Om GET-parameters mee te geven, gebruik html5::load_module_param()
*/
function load_modules() {
if (func_num_args() > 0) {
for ($i=0; $i<func_num_args(); $i++) {
return require_once( (string)func_get_arg($i) );
}
} else {
trigger_error('Geen parameters opgegeven',E_USER_NOTICE);
return false;
}
}
/**
* html5::load_module_param($module,$params)
* @category Model
* @package Modules
* @param $module constante, constante met prefix MOD_
* @param $params url-query, de GET-parameters
* @return false bij faling, main::require_once() bij succes
* @see html5::load_modules()
*
* Laad een module met de opgegeven GET-parameters.
* (Als je geen parameters opgeeft, gebruik dan html5::load_modules())
*/
function load_module_param($module,$params) {
if ($this->is_module($module)) {
return require_once( (string)$module.'?'.$params );
} else {
trigger_error($module,' is geen geldige module',E_USER_WARNING);
return false;
}
}
/**
* html5::include_add($file_path)
* @category Model/View
* @package Includes
* @param $file_path path, het pad naar de te includeren-bestand
* @return void
*
* Voegt het opgegeven bestand toe aan de buffer.
* Gebruik html5::include_view() om bestand te laten zien.
*/
function include_add($file_path) {
$this->aIncludeBestanden[count($this->aIncludeBestanden)] = $file_path;
}
/**
* html5::include_clear()
* @category allround
* @package Include
* @return void
*
* Leegt de buffer van html5::include_add().
*/
function include_clear() {
$this->aIncludeBestanden = array();
}
/**
* html5::include_view($output_only=true,$clear_cache)
* @category View
* @package Include
* @param $output_only bool, bepaalt of er alleen output moet worden verzonden. (standaard true)
* @param $clear_cache bool, bepaalt of na het uitvoeren de buffer met html5::include_clear() moet worden geleegd (standaard true)
* @return output
*
* Laat alle bestanden uit de buffer zien.
* Als $output_only op true staat, wordt readfile() gebruikt, anders include.
*/
function include_view($output_only=true,$clear_cache=true) {
foreach($this->aIncludeBestanden as $key=>$value) {
if ($output_only) {
readfile($value);
} else {
include $value;
}
}
if ($clear_cache) {
$this->include_clear();
}
}
/**
* html5::define_css($sPath)
* @category Model/View
* @package CSS
* @param $sPath path, pad naar te laden bestand.
* @return false bij falen en true bij succes
*
* Voegt het opgegeven pad toe aan de interne array.
* Met html5::toon_head() wordt de stylesheet echt geladen.
*/
function define_css($sPath) {
if ($this->bHeadView) {
trigger_error('Kan stylesheet niet laden, header is al weergegeven',E_USER_NOTICE);
return false;
} elseif (!file_exists($sPath)) {
trigger_error('Kan stylesheet niet laden, bestand niet gevonden',E_USER_NOTICE);
return false;
} else {
$this->aStyleSheets[count($this->aStyleSheets)] = $sPath;
return true;
}
}
/**
* html5::define_css_inline($sStyles,$replace)
* @category Model/View
* @package CSS
* @param $sStyle string(long), de styles die geladen moeten worden.
* @param $replace bool, bepaalt of de geladen css moet worden overschreven. (standaard false)
* @return void
*
* Voegt de opgegeven css toe aan de string die tussen de style-tags moet worden gezet.
*/
function define_css_inline($sStyles,$replace=false) {
if ($replace) {
$this->sInlineCSS = $sStyles;
} else {
$this->sInlineCSS .= $sStyles;
}
}
/**
* html5::clear_css()
* @category allround
* @package CSS
* @return void
*
* Zet de opgeslagen CSS terug naar NULL
*/
function clear_css() {
$this->sInlineCSS = NULL;
}
/**
* html5::define_javascript($sJavaScript,$loadJquery,$replace)
* @category Model
* @package JS
* @param $sJavaScript string(long), de te laden JavaScript.
* @param $loadJquery bool, geeft aan of het JQuery-framework moet worden geladen. (standaard false)
* @param $replace bool, geeft aan of de opgeslagen JavaScript moet worden overschreven.
* @return false bij faling, void bij succes
*
* Laad opgegeven JavaScript om tussen de script-tags te zetten bij html5::toon_head()
*/
function define_javascript($sJavaScript,$loadJquery=false,$replace=false) {
if ($this->bHeadView) {
trigger_error('Kan geen javascript toevoegen, header is al getoond',E_USER_NOTICE);
return false;
}
if ($loadJquery && $this->bJQueryLoad === false) {
$this->bJQueryLoad = true;
}
if ($replace) {
$this->sJavaScript = $sJavaScript;
} else {
$this->sJavaScript .= $sJavaScript;
}
}
/**
* html5::javascript_clear()
* @category allround
* @package JS
* @return void
*
* Verwijderd de opgeslagen JavaScript uit de buffer
*/
function javascript_clear() {
$this->sJavaScript = NULL;
}
/**
* html5::load_JQuery()
* @category allround
* @package JS
* @return void
*
* Zorgt dat bij html5::toon_head() het JQuery-framework wordt geladen.
*/
function load_JQuery() {
$this->bJQueryLoad = true;
}
}
class ravithrasworld extends html5 {
/***** INTERNAL PROPERTIES *****/
/**
* ravithrasworld::$aReport
* @var array() or NULL
* @access Wordt een aray bij ravithrasworld::report()
* @see ravithrasworld::report()
* @internal
*
* Bevat gegevens over de te raporteren problemen
*/
private $aReport = NULL;
/***** METHODS *****/
/**
* ravithrasworld::__construct($spTitel,$sCSSpad,$bIngelogd)
* @category Model
* @param $spTitel string(64), de titel van de webpagina
* @param $sCSSpad false|path, bij false wordt het standaard css-pad gebruikt en anders het opgegeven pad
* @return void
*
* Gebruikt html5::__construct() met een paar kleine aanpassingen:
* A) Aan de titel wordt ' | Ravithrasworld' toegevoegd
* B) De oorspronkelijke titel wordt ingekort tot 49 tekens
* C) Er wordt zonodig standaard css toegepast
* D) De css voor het menu wordt bijgevoegd
* E) De tijd-bibliotheek wordt geladen
*/
function __construct($spTitel,$sCSSpad=false) {
parent::__construct($spTitel);
$this->sTitel = substr($this->sTitel,0,49);
$this->sTitel .= ' | Ravithrasworld';
$this->load_modules(MOD_TIME);
if (!$sCSSpad) { //default
$this->define_css('SIS/opmaak.css');
} else {
$this->define_css($sCSSpad);
}
$this->define_css('SIS/menu.css');
}
/**
* ravithrasworld::toon_head($kop)
* @category View
* @param $kop string, toont een kop tussen h1-tags indien niet leeg (standaard leeg)
* @return output
*
* Gebruikt html5::toon_head() met een paar kleine aanpassingen:
* A) Het menu; het account venster en de kalender wordt bijgevoegd met html5::include_add()
* B) De standaard-header/master-head wordt ingevoegd
* C) Indien opgegeven wordt een kop geëcho't
*/
function toon_head($kop=null) {
parent::toon_head();
$this->include_add('SIS/menu.html');
$this->include_add('SIS/account.inc.php');
$this->include_add('SIS/kalender.inc.php');
$this->include_view(false);
$this->flush();
echo '<div id="banner" style="text-align:center;"><img src="plaatjes/banner.png" width="750" height="150" alt="banner"></div>';
echo '<div id="content">';
if (!empty($kop)) {
echo '<h1>',$kop,'</h1>';
}
}
/**
* ravithrasworld::__destruct()
* @category automatic
*
* Sluit de content af met '</div>' en voert dan html5::__destruct() uit.
* Als laatste worden eventuele report-meldingen verstuurd
*/
function __destruct() {
echo '</div>';
parent::__destruct();
$this->flush();
if (is_array($this->aReport)) {
$bericht = 'Er is een foutmelding uitgegaan via ravithrasworld::report().
Het bericht is:
"'.$this->aReport['bericht'].'"
Het type van de fout is: '.$this->aReport['type'].'
Het bestand waar de fout optrad: '.$this->aReport['bestand'];
mail('[email protected]','Foutmelding betreft: '.$this->aReport['type'],$bericht);
}
}
/**
* ravithrasworld::report($message,$type)
* @category Controller
* @package error
* @param $message string, het bericht dat verstuurd moet worden.
* @param $type string, type van de fout. (standaard database)
* @return void
*
* Zorgt dat bij het uitvoeren van ravithrasworld::__destruct() een melding naar de webmaster wordt gestuurd
*/
function report($message,$type='database') {
$this->aReport['bericht'] = $message;
$this->aReport['type'] = $type;
$this->aReport['bestand'] = $_SERVER['PHP_SELF'];
}
/**
* ravithrasworld::required_input()
* @category View
* @package error
* @return output
*
* Toont een gebruikersvriendelijke melding dat er GET-variabelen ontbreken.
* BREEKT SCRIPT AF!
*/
function required_input() {
$this->toon_head();
echo '<h1>Hmmm... er klopt iets niet</h1>';
echo 'Oeps, je bent vanaf de verkeerde plek hier gekomen!<br>';
echo 'Klik hierboven op een link om verder te gaan.<br><br>';
echo 'Blijft dit gebeuren?<br>';
echo '<a href="bugs.php?type=bug">Meld dan een bug</a>';
exit;
}
}
?>
Volgens mij krijg je een probleem doordat je het via een url probeert in te lezen (waarom een url en niet gewoon een pad?). Van de php.net pagina:
Remote file may be processed at the remote server (depending on the file extension and the fact if the remote server runs PHP or not) but it still has to produce a valid PHP script because it will be processed at the local server. If the file from the remote server should be processed there and outputted only, readfile() is much better function to use. Otherwise, special care should be taken to secure the remote script to produce a valid and desired code.
Als ik jouw url aanroep (gewoon in de browser) zie ik alleen de output van dat script, niet de content. Dat betekent dat jouw script dat ook ziet. Aangezien php gewoon een string krijgt via de include wordt die gewoon direct doorgestuurd naar het scherm, maar verder krijgt php niets te zien. Dat jij dus die test string op je scherm krijgt ligt niet aan de echo, maar aan het feit dat het voor php niet tussen php tags staat.
Oplossing 1: gewoon via een lokaal pad inlezen.
Oplossing 2: geen php extensie geven, maar een txt extensie (bijvoorbeeld), dan wordt de code niet al op de extern server uitgevoerd, maar gewoon als geheel doorgegeven. (Wat ik overigens heel erg zou afraden, want dan is je script publiekelijk te zien, door iedereen!!)
<div id="kalender">
<h3 style="color: red;">
<?php
echo parse_date(date('Y-m-d'));
?>
</h3>
Er is geen event en het is geen speciale dag.<br />
Jammer...
</div>
Een include doe je normaal gesproken via een lokaal pad. php opent dan het bestand, leest het in en parsed the code. Doe je het echter via een url, dan wordt het bestand uitgevoert op de externe server (zelfs als het op dezelfde server staat!) en krijg je dus het resultaat van het script terug, niet gewoon de code.
Roep de url van je te includen script maar eens aan in je browser. Wat je dan in je scherm ziet is wat php ook ziet. Dat is dus alleen de string, geen code. De functie bestaat dus niet. Die bestond wel voor een split second in de request waarin dat script werd opgevraagd, maar daar heb je niets aan.
Niet doen. Het is een groot beveiligingsrisico, iedereen kan dan namelijk je code zien.
Staan die websites op dezelfde server? Dan kunnen al die websites gewoon gebruik maken van hetzelfde pad.
