OXID Community Forum

> German forum > Entwickler > PHP7 Update-Paket für OXID eShop
Login
FAQ Members List Calendar Search Today's Posts Mark Forums Read

Thread: PHP7 Update-Paket für OXID eShop


Reply
 
Thread Tools Search this Thread Display Modes
#Top   #11  
Old 08-21-2017, 10:37 PM
vanilla thunder's Avatar
vanilla thunder vanilla thunder is online now
Marco's Sidekick
Join Date: Nov 2010
Posts: 6,454
vanilla thunder is on a distinguished road
Default Re: PHP7 Update-Paket für OXID eShop

das Problem ist, dass PHP bei Parse Errors keine Exception wirft, sondern ein ParseError Object.
Oxid will aber eine Exception haben.
Daher kommt nicht die eigentliche Fehlermeldung, sondern die Fehlermeldung, dass die eigentliche Fehlermeldung nicht verarbeitet werden konnte.
Man könnte sagen "OXID scheiterte beim Scheitern"

das hier könnte vorerst Abhilfe schaffen:
PHP Code:
public function handleUncaughtException($exception)
{
       if(!
$exception instanceof Exception$exception = new \Exception($exception);
.... 
Reply With Quote
#Top   #12  
Old 08-22-2017, 12:02 AM
OXID-Design's Avatar
OXID-Design OXID-Design is offline
Senior Member
Join Date: Mar 2013
Location: DE, Karlsruhe
Posts: 749
OXID-Design is on a distinguished road
Default Re: PHP7 Update-Paket für OXID eShop

Quote:
Originally Posted by Mitmacher View Post
Es dürften auch alles dieselben Howtos sein, aus denen "OXID-Design" dann auch das Update-Paket bastelte.
Nein, das ist doch nicht wahr. Schaut Ihr die Dateien nicht an die Ihr rüber kopiert? Es sind doch mindestens 5-6 Dateien mehr die ich extra angepasst habe.

@vanilla_thunder

Wenn ich mehr Zeit habe werde ich deine Änderung zuerst testen und mein Update-Paket dementsprechend aktualisieren.

Grüße
Rafig
__________________
OXID-Design, Module und Einsatzbereite Templates für OXID eShop
-----------------------------------------------------------------------------
OXIN Responsive | Coraline | MIGO | SAFA | Dinesh
Reply With Quote
#Top   #13  
Old 08-22-2017, 07:46 AM
patchwork.de patchwork.de is offline
Senior Member
Join Date: Jul 2012
Location: im Taunus
Posts: 1,055
patchwork.de is on a distinguished road
Default AW: PHP7 Update-Paket für OXID eShop

meiner Meinung nach sollten ParseErrors grundsätzlich beim Schreiben des Codes vermieden werden durch Verwendung eines entsprechenden Editors.
Darüberhinaus zeigt mir zB IonCube beim codieren auch (Parse)Errors an.
__________________
viele Grüße aus dem Taunus

www.patchwork.de
Version 4.10.5 - Flow mit child-theme

kostenloses WaWi:
wawi.patchwork.de
Reply With Quote
#Top   #14  
Old 08-22-2017, 08:22 AM
vanilla thunder's Avatar
vanilla thunder vanilla thunder is online now
Marco's Sidekick
Join Date: Nov 2010
Posts: 6,454
vanilla thunder is on a distinguished road
Default Re: PHP7 Update-Paket für OXID eShop

Da stimme ich durchaus zu, aber gerade in der professionellen Umgebung wird der Code für ein Modul oft maschinell aus mehreren Bestandteilen zusammengestellt und dann durch automatische Tests durchgejagt.
Gerade da wäre die echte Fehlermeldung statt dem Abrakadabra doch sehr sinnvoll, um schnell die Ursache des Problems festzustellen.
Reply With Quote
#Top   #15  
Old 08-22-2017, 08:38 AM
patchwork.de patchwork.de is offline
Senior Member
Join Date: Jul 2012
Location: im Taunus
Posts: 1,055
patchwork.de is on a distinguished road
Default AW: Re: PHP7 Update-Paket für OXID eShop

Quote:
Originally Posted by vanilla thunder View Post
... die echte Fehlermeldung statt dem Abrakadabra ....
... und die sehe ich in meiner Entwicklungsumgebung letztendlich im IonCube...
__________________
viele Grüße aus dem Taunus

www.patchwork.de
Version 4.10.5 - Flow mit child-theme

kostenloses WaWi:
wawi.patchwork.de
Reply With Quote
#Top   #16  
Old 08-22-2017, 11:57 AM
Mitmacher Mitmacher is offline
Senior Member
Join Date: Oct 2010
Posts: 408
Mitmacher is on a distinguished road
Default Re: PHP7 Update-Paket für OXID eShop

Hm, okay, dann bin ich halt unprofessionell, da ich nicht IonCube nutze und keine aufgeblähte Entwicklungsumgebung, aber ist das nicht meine legitime Entscheidung? Außerdem ist die OXID-Zukunft doch angeblich unverschlüsselt und es wird uns Entwicklern sogar empfohlen, diese Entscheidung mitzutragen, also bin ich der Zeit doch eigentlich quasi um Jahre voraus!?
Und @oxid-design:
Quote:
Nein, das ist doch nicht wahr. Schaut Ihr die Dateien nicht an die Ihr rüber kopiert?
Sorry, dass ich dich in einem Topf warf, aber es ist nun mal so, dass ALLE Howtos identisch sind, also lag der Verdacht schon nahe, dass du auch dazugehörst, nur halt netterweise ein ZIP-Paket geschnürt hattest. Danke dafür, keine Frage, nur wundere dich bitte nicht, dass man es nicht explizit testet, wenn man schon dutzend Mal (vermeintlich) dasselbe gelesen hat und (wie erwähnt) halt wg. der nervigen Social-Buttons es gar nicht erst downloaden darf. Und nein, ich bin da als Facebook-Verweigerer nicht hinterher, sondern wiederum der Zeit voraus, wie ich finde…
Aber egal, zurück zum Thema, prima Kommentar:
Quote:
Man könnte sagen "OXID scheiterte beim Scheitern"
Und danke für die Lösungs-Idee! Es will aber trotzdem nicht, und ich verstehe die Logik so: wenn $exception NICHT instanceof Exception ist, dann mache es zu einer solchen Instanz. Damit bleibt das Problem doch bestehen, denn ein parseError ist eben nicht mehr "kompatibel", oder wie? Sorry, leider immer noch Bahnhof…
Reply With Quote
#Top   #17  
Old 08-22-2017, 12:57 PM
patchwork.de patchwork.de is offline
Senior Member
Join Date: Jul 2012
Location: im Taunus
Posts: 1,055
patchwork.de is on a distinguished road
Default AW: Re: PHP7 Update-Paket für OXID eShop

Quote:
Originally Posted by Mitmacher View Post
Hm, okay, dann bin ich halt unprofessionell, da ich nicht IonCube nutze und keine aufgeblähte Entwicklungsumgebung, aber ist das nicht meine legitime Entscheidung? Außerdem ist die OXID-Zukunft doch angeblich unverschlüsselt und es wird uns Entwicklern sogar empfohlen, diese Entscheidung mitzutragen, also bin ich der Zeit doch eigentlich quasi um Jahre voraus!?
IonCube ist für uns nur der Gürtel zum Hosenträger.
Das wichtigste ist die Entwicklungsumgebung. Ohne irgendwas gescheites wie zB NetBean kommt man - insbesondere bei PHP7 - nicht weit
__________________
viele Grüße aus dem Taunus

www.patchwork.de
Version 4.10.5 - Flow mit child-theme

kostenloses WaWi:
wawi.patchwork.de
Reply With Quote
#Top   #18  
Old 08-22-2017, 01:26 PM
Mitmacher Mitmacher is offline
Senior Member
Join Date: Oct 2010
Posts: 408
Mitmacher is on a distinguished road
Default Re: PHP7 Update-Paket für OXID eShop

Ja gut, ich bezog mich auch nur auf die Verschlüsselung. Und ganz ohne Entwicklungsumgebung arbeite ich natürlich auch nicht, nur ist das über die Jahre sehr individuell gewachsen und der Kern bleibt "nur" ein guter Editor wie UltraEdit. Der kann sogar Syntax-Prüfung, die ich aber bisher nie brauchte, weil PHP immer sagte, wo es genau klemmt. Nun aber nicht mehr, also muss ich mich evtl. umgewöhnen, okay. Wenn es auch nur dieser eine Punkt wäre, könnte ich da wohl mit leben, aber allmählich wächst einem OXID echt über den Kopf. Es wäre interessant, ob das nur mir so geht (also als Einzelkämpfer, nicht als Agentur) aber das wäre hier zu off-topic, um es zu vertiefen. Jedenfalls hat man bis dato halt eigentlich nur einen Editor und ein OXID-ZIP gebraucht und konnte loslegen, wenn man PHP beherrschte (na ja, etwas vereinfacht). Allmählich braucht es wohl ein 10-jähriges Informatik-Studium und Spezialwissen in zig Tools, nur um einen Demo-Shop (v6) zum Laufen zu bringen! Wenn das so weitergeht, bleibt einem fast nichts anderes als aufzugeben, denn vor lauter Administration bleibt leider keine Zeit mehr, um die eigentliche Arbeit zu erledigen die auch Geld einbringt…
Reply With Quote
#Top   #19  
Old 08-22-2017, 01:38 PM
vanilla thunder's Avatar
vanilla thunder vanilla thunder is online now
Marco's Sidekick
Join Date: Nov 2010
Posts: 6,454
vanilla thunder is on a distinguished road
Default Re: PHP7 Update-Paket für OXID eShop

Composer und die ganze andere Grütze ist nur für Entwickler und Agenturen gedacht. Normale Shopbetreiber bekommen weiterhin eine fertige Zip zum Hochladen, zumindest laut Marco.
Reply With Quote
#Top   #20  
Old 08-22-2017, 02:27 PM
wolkenkrieger wolkenkrieger is offline
Senior Member
Join Date: May 2014
Posts: 382
wolkenkrieger is on a distinguished road
Default Re: PHP7 Update-Paket für OXID eShop

@patchwork

Ich arbeite auch ohne IonCube (und setze auch keine verschlüsselten Module ein ... aus Gründen) aber sowohl mit netbeans als auch mit PHPStorm ... trotzdem hätte ich es gerne, wenn mir mein System Fehler meldet - und zwar unabhängig von der IDE.

Und die Intention von mitmacher, mit einfachsten Mitteln PHP und damit auch Oxid entwickeln zu können, ist so verkehrt nicht.
__________________
Shopversion: CE 4.10.3
Theme: vt's Glow
Reply With Quote
Reply

Bookmarks

Tags
eshop, php7, updatepaket

« Previous Thread | Next Thread »
Thread Tools Search this Thread
Search this Thread:

Advanced Search
Display Modes

Nicht Sichtbar
Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

Similar Threads
Thread Thread Starter Forum Replies Last Post
Oxid auf PHP7 Installieren / Einrichten Akila Installation und Konfiguration 5 09-25-2016 01:28 PM
Update von OXID eShop CE 4.5.0 auf 4.5.1 webtek Installation und Konfiguration 3 08-10-2011 04:54 PM
OXID eSHOP Update von 4.4.7 auf 4.4.8 Problem beim Datenbank-Update webtek Installation und Konfiguration 5 04-06-2011 06:58 PM
Welche Update-Paket Quan Installation und Konfiguration 16 07-08-2010 01:15 PM
update paket jonnijuliani Installation und Konfiguration 9 05-25-2010 04:54 PM

All times are GMT +1. The time now is 07:35 PM.