[GELÖST] Fehler /lib/misc.functions.php on line 1590

Henk1060 · 05. Januar 2012 14:05

Heute ist echt nicht mein tag.... wink schon wieder was *G*

bekomme diesen Fehler-- da ich schon wieder was mit dem modul ShopMadesimple erstellen soll...
Ich denke der fehler kommt vom Calguys Modul-Erweiterungen könnte das sein?

Hier klicken, um den Code zum Kopieren zu markieren

Warning: Parameter junk is not known by module ShopMadeSimple dropped in /lib/misc.functions.php on line 1590

NaN · 05. Januar 2012 14:20

Lass mich raten: Der Fehler kommt nur im Frontend und auch nur bei Pretty-Urls?

Henk1060 · 05. Januar 2012 14:22

jeah... also dürfte das schon was bekanntes sein... habe zwar google gefragt... jedoch keine guten Ergebnisse

NaN · 05. Januar 2012 14:49

Naja, zumindest die Fehlermeldung "Parameter X is not known by module Y" ist ein bekanntes Problem und betrifft viele Module. Und im Zusammenhang mit "junk" kam ich auf die Idee mit den Pretty-URLs, weil da oftmals auch eine Menge Müll mit in der URL steht, der keine weitere Funktion hat, als hübsch auszusehen.

Das Problem tritt meistens dann auf, wenn verschiedene Module miteinader zusammenarbeiten sollen oder wenn beim Programmieren vergessen wurde, einen Parameter als erlaubten Parameter zu definieren. Dabei kann es passieren, dass Parameter an ein Modul übergeben werden, die es nicht kennt. Mein Feature-Request bezüglich einer Funktion, mit der Entwickler die erlaubten Parameter eines Moduls ermitteln können, um nur korrekte Parameter zu übergeben, wurde mit einer ziemlich bekloppten Begründung abgewiesen.

Dir wird an dieser Stelle nichts anderes übrig bleiben, als die Ausgabe von Fehlermeldungen (z.B. via .htaccess) zu unterdrücken, oder jedes Modul, welches diese Meldung verursacht, im Quelltext um den jeweiligen Parameter zu erweitern.

Henk1060 · 05. Januar 2012 15:16

naja das finde ich aber Doof von diesen Lieben Entwicklern!!!
Das heißt im klar text das das jetzt öfters vorkommen kann... den in der alten Version hatte ich diesen Fehler nicht.

Ok: dann ist die einfachste lösung .htaccess denke ich mal
Habe so eben versucht mit diesen teil hier
#php_flag display_startup_errors Off
#php_flag display_errors Off
#php_flag html_errors Off

jedoch ohne erfolg... oder muss man bei CMSMS auch noch auf was anderes achten

NaN · 05. Januar 2012 15:22

Die Raute-Zeichen am Zeilenanfang müssen weg.
Sonst sind es nur Kommentarzeilen.

Henk1060 · 05. Januar 2012 15:30

ja es wird wirklich zeit für eine brille für mich danke nochmals...
jep nur falsch kopiert... raute ist weg! jedoch immer noch mit der fehler ausgabe

NaN · 05. Januar 2012 15:35

Hilft diese Zeile?

Hier klicken, um den Code zum Kopieren zu markieren

php_value error_reporting 0

Henk1060 · 05. Januar 2012 15:47

Hier die fast noch jungfräuliche .hat
leider kommt des fehler ausgabe noch immer

Hier klicken, um den Code zum Kopieren zu markieren

# Attempt to override some php settings, these settings may be helpful on some hosts if your
# default configuration does not meet CMS's minimum requirements, and your host
# has given your account appropriate permissions
#php_value upload_max_filesize "10M"
#php_value session_save_path "tmp/cache"

#php_flag magic_quotes_gpc Off
#php_flag register_globals Off
#php_flag session.use_trans_sid Off

# (this is important, so uncomment if your host permit)
#Options -Indexes
#ServerSignature Off

#Options +FollowSymLinks

# To prevent E_STRICT problems with PHP 5.3+ you can uncomment the following lines
# Note: These settings should only be enabled for production sites!
php_flag display_startup_errors Off  
php_flag display_errors Off  
php_flag html_errors Off 
php_value docref_root 0
php_value docref_ext 0
php_value error_reporting 0

<IfModule mod_rewrite.c>
RewriteEngine on
#
#Sub-dir e.g: /cmsms
RewriteBase /

# 301 Redirect all requests that don't contain a dot or trailing slash to
# include a trailing slash
# but ignore POST requests.
#RewriteCond %{REQUEST_URI} !/$
#RewriteCond %{REQUEST_URI} !\.
#RewriteCond %{REQUEST_METHOD} !POST$
#RewriteRule ^(.*) %{REQUEST_URI}/ [R=301,L]

# Rewrites urls in the form of /parent/child/
# but only rewrites if the requested URL is not a file or directory
#
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.+)$ index.php?page=$1 [QSA]
</IfModule>

<IfModule mod_header.c>
# Disable ETags
Header unset ETag
FileEtag None
</IfModule>

<IfModule mod_deflate.c>
# Compress css, plaintext, xml, gif, and images in transport.
AddOutputFilterByType DEFLATE text/css text/plain text/xml image/gif image/jpeg image/png
</IfModule>

<IfModule mod_expires.c>
<FilesMatch "\.(ico|pdf|flv|jpg|jpeg|png|gif|js|css|swf)$">
# Set expires tags on various files... so that the browser wont attempt to reload them.
ExpiresActive On
ExpiresDefault "access plus 1 year"
<IfModule mod_header.c>
  # Setting cache control to public allowes proxy servers to cache the items too.
  Header set Cache-Control "public"
</IfModule>
</FilesMatch>
</IfModule>

Beitrag geändert von Henk1060 (05. Januar 2012 16:08)

piratos · 05. Januar 2012 16:50

NaN schrieb:

wurde mit einer ziemlich bekloppten Begründung abgewiesen.

Ist sie nicht das CG quasi eine Anleitung gibt wie man es machen kann, wenn man als Entwickler es anbieten möchte bzw. absichern möchte.
Offenbar aber hat er selbst kein Interesse daran.

Über htaccess lässt sich so etwas nicht ändern .

Henk1060 · 05. Januar 2012 16:53

ändern zwar nicht aber die Meldung ausblenden oder irgendwie diesen Fehler zu beheben.
wäre schon ein Hit wink )

piratos · 05. Januar 2012 16:58

Dazu muss man nur die Zeile 1590 auskommentieren und dann ist es erledigt.

Henk1060 · 05. Januar 2012 17:09

ok dann ist das diese zeile

Hier klicken, um den Code zum Kopieren zu markieren

 {
          trigger_error('Parameter '.$key.' is not known by module '.$modulename.' dropped',E_USER_WARNING);
          continue;
        }

Ist für E_USER_WARNING die fehler ausgabe zuständig oder?

ok und wie muss ich da jetzt weiter vorgehen?

Edit: habs mit error_reporting(0) geschaft ist das dir richtige lösung ?

Beitrag geändert von Henk1060 (05. Januar 2012 17:12)

piratos · 05. Januar 2012 17:17

Auskommentieren

da macht man vor einer einzigen Zeile einfach ein

#

davor.

error_reporting(0) ist ein Gewaltmittel, setzt man das ein erkennt man möglicherweise nie warum es an einer Stelle nicht läuft.

Henk1060 · 05. Januar 2012 17:24

OK! Es muss ja nicht gleich die ganz Harte Variante sein, aber sie Funkt. wink )
Jep auch mit der Raute ist natürlich eine sehr gute Lösung.... Danke nochmals für die Hilfestellung

NaN · 05. Januar 2012 18:48

piratos schrieb:

Ist sie nicht das CG quasi eine Anleitung gibt wie man es machen kann, wenn man als Entwickler es anbieten möchte bzw. absichern möchte.

Diese "Anleitung" ist aber für'n A... weil er das Problem überhaupt nicht versteht.
Seine "Anleitung" basiert auf der Idee, dass der Anwender bereits beim Aufruf eines Moduls alle Parameter, die einem anderen Modul übergeben werden sollen, kennen und angeben muss. Und das ist einfach "bekloppt".

Genausogut ist es bekloppt, sich als Entwickler immer und immer wieder den Quelltext anderer Module anschauen zu müssen, um zu wissen, welche Parameter es gibt und welche nicht. Module, die miteinander arbeiten sollen, müssen wissen, welche Parameter erlaubt sind, und welche nicht. Eine API Funktion mit einer Zeile Code tut niemandem weh und macht da vieles sehr viel einfacher. Es gibt ja nichtmal eine Möglichkeit die eigenen Modulparameter zu ermitteln außer, sie selbst nochmal in einer Membervariable zu speichern. Und diese Dopplung halte ich ebenfalls für bekloppt.

Ich hatte schon oft die Situation, dass ich ein Array mit Parametern, die nur teilweise auch für ein anderes Modul galten, an eine Modulfunktion übergeben wollte. Man müsste jetzt "von Hand" jeden einzelnen Eintrag aus dem Array entfernen, der nicht für das andere Modul gilt oder ein zweites Array aufbauen. Eleganter wäre es, wenn man als Entwickler einfach eine Funktion hätte, mit der solche Sachen automatisiert ablaufen können.
Bsp.:

Hier klicken, um den Code zum Kopieren zu markieren

$module = &cms_utils::get_module('Module X');
$params = array_intersect_assoc($my_params, $module->GetAllowedParameters());
$module->DoAction('action', $id, $params, $returnid);

Somit müsste man sich überhaupt keine Platte mehr machen, welche Parameter es im anderen Modul gibt und wie diese heißen. Es würde immer ein "sauberes" Array mit erlaubten Parametern an das andere Modul übergeben werden und diese ebenfalls völlig bekloppte Fehlermeldung "Parameter X not known by module Y" wäre auf ein Minimum reduziert.

piratos · 05. Januar 2012 20:36

Also auf die Schnell sticht mir das hier ins Auge

Hier klicken, um den Code zum Kopieren zu markieren

[== PHP ==]
/**
     * Returns a list of parameters and their help strings in a hash.  This is generally
     * used internally.
     *
     * @final
     * @internal
     * @access private
     * @return array
     */
    final public function GetParameters()

damit lässt sich doch wohl etwas anfangen ?

NaN · 05. Januar 2012 21:47

Ist nicht ganz das was ich meinte. Diese Funktion gibt die Parameter inklusive Hilfetext in der Form wieder:

Hier klicken, um den Code zum Kopieren zu markieren

array(
    'name' => foo
    'default' => bar
    'helptext' => ...
    'optional' => true/false
)

Besser wäre einfach ein Array mit den Namen der Parameter.
Z.B. die Membervariable $param_map bzw. nur deren Schlüssel.
Aber wenn es sonst nichts gibt, muss das wohl reichen.

Forum für CMS/made simple

#1 05. Januar 2012 14:05

[GELÖST] Fehler /lib/misc.functions.php on line 1590

#2 05. Januar 2012 14:20

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#3 05. Januar 2012 14:22

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#4 05. Januar 2012 14:49

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#5 05. Januar 2012 15:16

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#6 05. Januar 2012 15:22

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#7 05. Januar 2012 15:30

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#8 05. Januar 2012 15:35

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#9 05. Januar 2012 15:47

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#10 05. Januar 2012 16:50

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#11 05. Januar 2012 16:53

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#12 05. Januar 2012 16:58

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#13 05. Januar 2012 17:09

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#14 05. Januar 2012 17:17

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#15 05. Januar 2012 17:24

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#16 05. Januar 2012 18:48

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#17 05. Januar 2012 20:36

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

#18 05. Januar 2012 21:47

Re: [GELÖST] Fehler /lib/misc.functions.php on line 1590

Fußzeile des Forums