Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

$_REQUEST-Nutzung aufgelöst #123

Merged
merged 3 commits into from
Sep 17, 2024
Merged

$_REQUEST-Nutzung aufgelöst #123

merged 3 commits into from
Sep 17, 2024

Conversation

christophboecker
Copy link
Member

Die von RexStan monierte Nutzung von $_REQUEST (if(0 < count($_REQUEST))...) ist durch die Abfrage des Console-Objektes abgelöst. Abhängig von der Frage wurde dem Addon-Titel in der Navigation ein Plus-Button angehängt.

Zu weiteren Einzelheiten des Wie und Warum siehe hier.

Der Code dazu wurde nach Neues::epPagesPrepared verlagert. Die boot.php wird dadurch schlanker und übersichtlcher. Nebeneffekt: per Saldo wird weniger Code via boot.php compiliert und abgearbeitet, da die meisten Seitenaufrufe i.d.R. via FE und nicht BE erfolgen.

RexStan ist einverstanden

@alxndr-w alxndr-w merged commit ca995cf into main Sep 17, 2024
@alxndr-w alxndr-w deleted the christophboecker-patch-1 branch September 17, 2024 17:56
@alxndr-w
Copy link
Member

Perfekt, danke für die Intervention. Der EP ist eine elegante Lösung und besser als das urspr. Konstrukt.

@alxndr-w
Copy link
Member

Error: Call to a member function setTitle() on null
File: src\addons\neues\lib\neues.php
Line: 104

Stacktrace
Function File Line
FriendsOfRedaxo\Neues\Neues::epPagesPrepared src\core\lib\extension.php 52
rex_extension::{closure} src\core\lib\util\timer.php 62
rex_timer::measure src\core\lib\extension.php 42
rex_extension::registerPoint src\core\backend.php 218
require src\core\boot.php 157
require public\redaxo\index.php 12
System report (REDAXO 5.17.1, PHP 8.3.7, MariaDB 10.11.9)
REDAXO
Version 5.17.1
PHP
Version 8.3.7
OPcache no
Xdebug no
Database
Version MariaDB 10.11.9
Character set utf8mb4
Server
OS WINNT
SAPI apache2handler
Webserver Apache/2.4.59
Request
Browser Chrome/128.0.0.0
Protocol HTTP/1.1
HTTPS yes
Packages
2factor_auth 2.2
accessdenied 2.1.1
adminer 2.0.2
auto_delete 1.0.2
backup 2.9.1
be_password 2.0.1
be_style 3.4.1
be_style/customizer 3.3.1
be_style/redaxo 3.4.1
bloecks 4.0.2
bloecks/cutncopy 4.0.2
cronjob 2.11.0
developer 3.9.2
iconpicker 2.0.0
install 2.12.1
mailer_profile 1.3.2
maintenance 3.2.1
mblock 3.4.13
media_manager 2.16.0
media_manager_responsive 2.0.0-dev-2024-09-14
mediapool 2.14.0
metainfo 2.11.0
mform 8.0.3
neues 6.0.0-dev-2024-09-16
phpmailer 2.13.1
plus_bs5 0.1.5
product 3.0.0
project dev
project_manager 1.5.3
project_manager/client 1.1.0
qanda 2.2.2
quick_navigation 8.1.3
redactor 2.5.2
respondo 0.0.1-dev-2024-04-07
rexstan 1.0.188
schwooops 0.0.1-dev-2024-08-26
search_it 6.11.2
search_it/documentation 6.10.1
speed_up 1.4.0
sprog 1.5.1
staff 1.4.1
stellenangebote 3.0.0-beta2
structure 2.17.1
structure/content 2.17.1
thumb 1.0.0-dev-2024-09-09
uploader 2.4.2
url 2.1.0
users 2.11.0
wenns_sein_muss 5.1.0
wildcard 1.0.0
ycom 4.3.0
ycom/auth 4.3.0
ycom/group 4.3.0
ydeploy 2.0.2
yform 4.2.1
yform/email 4.2.1
yform/manager 4.2.1
yform/rest 4.2.1
yform_field 2.7.1
yform_geo_osm 1.2.9
yform_quick_edit 1.0.1
yform_rapidmail 1.0.3
yform_spam_protection 1.2.6-dev
yform_usability 2.1.4
ymca 2.2.0
yrewrite 2.10.0
yrewrite_metainfo 2.1.1

@christophboecker
Copy link
Member Author

Kann ich nicht nachvollziehen; läuft bei mir. Ist auch unlogisch, denn der EP wird ja nur angezogen, wenn im Backend, boot.php abgearbeitegn (also Addon verfügbar) und mit Navigation (also auch die Addon-Seiten verfügbar.) Ja nun: Lösung wäre dennoch abzufragen, ob die Seite existiert. Läßt sich ja fix einbauen.

@alxndr-w
Copy link
Member

Ich war auch überrascht und kann jetzt nicht sagen, was zwischen deiner und meiner Installation unterschiedlich war. Wenn ich Zeit habe, schaue ich mir das nochmal in Ruhe an.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants