From 37a30434bb4f26c90ab56d9d4fde0ad0588422ab Mon Sep 17 00:00:00 2001 From: Hypolite Petovan Date: Mon, 16 Aug 2021 05:47:29 -0400 Subject: [PATCH] Remove remaining mentions of $a->argc/argv - This was breaking the layout of specific pages in Frio --- doc/Addons.md | 12 ++++----- doc/de/Addons.md | 13 +++++---- view/theme/frio/php/default.php | 2 +- view/theme/frio/php/standard.php | 46 +++++++++++++++----------------- 4 files changed, 37 insertions(+), 36 deletions(-) diff --git a/doc/Addons.md b/doc/Addons.md index a1c2c03ce2..4a32dc09f4 100644 --- a/doc/Addons.md +++ b/doc/Addons.md @@ -158,13 +158,13 @@ Addons may also act as "modules" and intercept all page requests for a given URL In order for a addon to act as a module it needs to declare an empty function `_module()`. If this function exists, you will now receive all page requests for `https://my.web.site/` - with any number of URL components as additional arguments. -These are parsed into an array $a->argv, with a corresponding $a->argc indicating the number of URL components. -So `https://my.web.site/addon/arg1/arg2` would look for a module named "addon" and pass its module functions the $a App structure (which is available to many components). -This will include: - +These are parsed into the `App\Arguments` object. +So `https://my.web.site/addon/arg1/arg2` would give this: ```php -$a->argc = 3 -$a->argv = array(0 => 'addon', 1 => 'arg1', 2 => 'arg2'); +DI::args()->getArgc(); // = 3 +DI::args()->get(0); // = 'addon' +DI::args()->get(1); // = 'arg1' +DI::args()->get(2); // = 'arg2' ``` To display a module page, you need to declare the function `_content(App $a)`, which defines and returns the page body content. diff --git a/doc/de/Addons.md b/doc/de/Addons.md index a0ab58de8e..276f5ed1b6 100644 --- a/doc/de/Addons.md +++ b/doc/de/Addons.md @@ -61,11 +61,14 @@ Addons können auch als "Module" agieren und alle Seitenanfragen für eine besti Um ein Addon als Modul zu nutzen, ist es nötig, die Funktion "addon_name_module()" zu definieren, die keine Argumente benötigt und nichts weiter machen muss. Wenn diese Funktion existiert, wirst du nun alle Seitenanfragen für "http://example.com/addon_name" erhalten - mit allen URL-Komponenten als zusätzliche Argumente. -Diese werden in ein Array $a->argv geparst und stimmen mit $a->argc überein, wobei sie die Anzahl der URL-Komponenten abbilden. -So würde http://example.com/addon/arg1/arg2 nach einem Modul "addon" suchen und seiner Modulfunktion die $a-App-Strukur übergeben (dies ist für viele Komponenten verfügbar). Das umfasst: - - $a->argc = 3 - $a->argv = array(0 => 'addon', 1 => 'arg1', 2 => 'arg2'); +Diese werden in das App\Arguments Objekt geparst. +So würde `http://example.com/addon/arg1/arg2` dies ergeben: +```php +DI::args()->getArgc(); // = 3 +DI::args()->get(0); // = 'addon' +DI::args()->get(1); // = 'arg1' +DI::args()->get(2); // = 'arg2' +``` Deine Modulfunktionen umfassen oft die Funktion addon_name_content(App $a), welche den Seiteninhalt definiert und zurückgibt. Sie können auch addon_name_post(App $a) umfassen, welches vor der content-Funktion aufgerufen wird und normalerweise die Resultate der POST-Formulare handhabt. diff --git a/view/theme/frio/php/default.php b/view/theme/frio/php/default.php index b2af772e61..d474a4d784 100644 --- a/view/theme/frio/php/default.php +++ b/view/theme/frio/php/default.php @@ -125,7 +125,7 @@ $is_singleuser_class = $is_singleuser ? "is-singleuser" : "is-not-singleuser";
'; if (!empty($page['content'])) { echo $page['content']; diff --git a/view/theme/frio/php/standard.php b/view/theme/frio/php/standard.php index ee90bea7d6..e4156d7460 100644 --- a/view/theme/frio/php/standard.php +++ b/view/theme/frio/php/standard.php @@ -22,6 +22,8 @@ use Friendica\DI; +$frio = 'view/theme/frio'; + ?> @@ -30,18 +32,16 @@ use Friendica\DI; - - - + - + t('Skip to main content'); ?> get('config','sitename'), - str_replace("~system.banner~", DI::config()->get('system','banner'), + echo str_replace('~config.sitename~', DI::config()->get('config','sitename'), + str_replace('~system.banner~', DI::config()->get('system','banner'), $page['nav'] ));}; ?> @@ -50,24 +50,24 @@ use Friendica\DI;
- "; if(!empty($page['aside'])) echo $page['aside']; echo" - "; if(!empty($page['right_aside'])) echo $page['right_aside']; echo" - "; include('includes/photo_side.php'); echo" + echo ' + -
-
-
argv[0]; echo "-content-wrapper\"> -
"; - if(!empty($page['content'])) echo $page['content']; echo" -
+
+
+
+
'; + if(!empty($page['content'])) echo $page['content']; echo' +
- "; + '; ?>
@@ -76,11 +76,10 @@ use Friendica\DI;