search

OpenWebconcept / plugin-openagenda-base

This plugin adds Events and Locations to WordPress which can be retrieved via the OpenAgenda REST API.
European Union Public License 1.2
0 stars 0 forks source link

Using namespaces #1

Closed mvdhoek1 closed 8 months ago

mvdhoek1 commented 8 months ago

Beschrijf de bug 'Admin' in de namespaces in class-plugin-php bevatten paden met hoofdletters en zonder hoofdletters:

use Openagenda_Base_Plugin\admin\Cmb2;
use Openagenda_Base_Plugin\Admin\Event_Dates;

Bij ons op de server gaat dit sowieso fout omdat de namespaces hoofdlettergevoelig zijn.

Te reproduceren N.v.t.

Verwacht gedrag Van use Openagenda_Base_Plugin\Admin\Event_Dates; naar use Openagenda_Base_Plugin\admin\Event_Dates;

Schermafbeeldingen N.v.t. -> https://github.com/OpenWebconcept/plugin-openagenda-base/blob/main/src/class-plugin.php#L18

Desktop (vul de volgende informatie in): N.v.t.

Smartphone (vul de volgende gegevens in): N.v.t.

Aanvullende context N.v.t.

richardkorthuis commented 8 months ago

Volgens de WP Coding Standards hoort dit juist wel in hoofdletters: https://developer.wordpress.org/coding-standards/wordpress-coding-standards/php/#namespace-declarations Dat er op enkele plaatsen dus geen hoofdletters werden gebruikt was fout en heb ik aangepast.

Ik zie echter niet in waarom dit op jullie server voor problemen zou zorgen, want de meegeleverde autoloader ze de namespace juist om in een lowercase directorystructuur: https://github.com/OpenWebconcept/plugin-openagenda-base/blob/main/src/class-autoloader.php#L38

mvdhoek1 commented 8 months ago

Top, bedankt voor het oplossen!