Mailer-Project Bug-Tracker - Branch 0.2.1-FINAL
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0000040Branch 0.2.1-FINALBugpublic2008-12-29 21:072016-11-23 11:28
Reporteruser13 
Assigned ToQuix0r 
PriorityurgentSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformApache 2.2.9 - mod_php 5.2.6OSWindows Vista Ultimate x64 SP1OS Version6001 SP1
Product Version0.2.1-FINAL 
Target Version0.2.1-FINALFixed in Version0.2.1-FINAL 
Summary0000040: Fehlermeldung bei Surfbar URL-Ändern
DescriptionBei Surfbar URL-Ändern tritt folgender Fehler auf, wenn man keinen Eintrag gewählt hat.

# __errorHandler(8, Undefined index: id, E:\InternetProjekte\mailtausch\inc\modules\admin\what-list_surfbar_urls.php, 50,
.......
 called at [E:\InternetProjekte\mailtausch\inc\modules\admin\what-list_surfbar_urls.php:50]
# include(E:\InternetProjekte\mailtausch\inc\modules\admin\what-list_surfbar_urls.php) called at [E:\InternetProjekte\mailtausch\inc\modules\admin\action-surfbar.php:47]
# include(E:\InternetProjekte\mailtausch\inc\modules\admin\action-surfbar.php) called at [E:\InternetProjekte\mailtausch\inc\modules\admin\admin-inc.php:352]
# ADMIN_DO_ACTION(list_surfbar_urls) called at [E:\InternetProjekte\mailtausch\inc\modules\admin.php:368]
# include(E:\InternetProjekte\mailtausch\inc\modules\admin.php) called at [E:\InternetProjekte\mailtausch\modules.php:157]
TagsNo tags attached.
Relationships
Attached Files

Notes
(0000094)
user13   
2008-12-29 21:13   
(Last edited: 2010-05-28 19:28)
Check ob $_POST['id'] vorhanden ist hinzugefügt. Behebt auch andere Fehlermeldungen wenn keine Auswahl gemacht worden ist.

http://www.mxchange.stelzi.net.invalid/patches/what-list_surfbar_urls-Fix.patch
(0000203)
Quix0r   
2009-02-18 23:48   
Hab einen allgemeinen if()-Block zuvor eingebaut und ist beim naechsten Commit dabei.

Auf die Art, wie du es loesen wolltest, wollte ich es anfangs auch, habe mich dann den Code schmaler zu halten umentschieden.

Tipp am Rande:
if (isset($_POST['delete']) && isset($_POST['id']))
schreibe ich nicht, eher:
if ((isset($_POST['delete'])) && (isset($_POST['id'])))
(0000206)
user13   
2009-02-19 13:32   
Ja das mit den hunderttausend Klammern in deinem Code ist mir bereits aufgefallen. Manche PHP-Programmierer würden das so werten wie zb. if ($bool == true) oder if ($bool == false) anstatt if ($bool) oder if (!$bool). Ich zb sehe da bei deiner Version vor lauter Klammern den Innhalt nicht mehr ;-) Da ja schon etliche Typos mit deiner Version reingekommen sind würde ich sagen, dass meine Version auch nicht verkehrt ist. Bei If-Statements nutze ich nur dann zusätzliche Klammern wenns wirklich verwirrend werden könnte und dann mache ich eher jeden Abgleich in eine eigene Zeile (auch besser bei der Anzeige eines Fehler -- da weis man sofort welcher Abgleich den Fehler produziert)
(0000207)
user13   
2009-02-19 13:33   
Ja ist mit der aktuellen Rev behoben.
(0000810)
Quix0r   
2016-11-23 11:28   
Und zu damit.

Issue History
Date ModifiedUsernameFieldChange
2008-12-29 21:07user13New Issue
2008-12-29 21:13user13Note Added: 0000094
2009-02-06 02:42Quix0rFixed in Version => 0.2.1-FINAL
2009-02-09 12:30user13Note Edited: 0000094
2009-02-10 19:42user13Prioritynormal => urgent
2009-02-18 23:48Quix0rNote Added: 0000203
2009-02-18 23:48Quix0rAssigned To => Quix0r
2009-02-18 23:48Quix0rStatusnew => assigned
2009-02-18 23:48Quix0rProjectionnone => tweak
2009-02-18 23:48Quix0rETAnone => < 1 day
2009-02-18 23:48Quix0rTarget Version => 0.2.1-FINAL
2009-02-18 23:48Quix0rDescription Updated
2009-02-19 13:32user13Note Added: 0000206
2009-02-19 13:33user13Note Added: 0000207
2009-02-19 13:33user13Statusassigned => resolved
2009-02-19 13:33user13Resolutionopen => fixed
2009-11-25 22:55Quix0rProduct Version0.2.1-FINAL =>
2009-11-25 22:55Quix0rBuildRev 708 => SVN-708
2010-05-28 18:59Quix0rProduct Version => 0.2.1-FINAL
2010-05-28 19:26Quix0rDescription Updatedbug_revision_view_page.php?rev_id=54#r54
2010-05-28 19:28Quix0rNote Edited: 0000094bug_revision_view_page.php?bugnote_id=94#r72
2016-11-23 11:28Quix0rNote Added: 0000810
2016-11-23 11:28Quix0rStatusresolved => closed