LAN_PLUGIN_NEWS_NAME

Alex, vr feb. 19 2021, 09:47A.M.

Erik wees me op het ontbreken van een taalstring op https://e107.nl/nieuws

Vandaag even gecontroleerd maar er staat nog niets in de engelse bestanden (geen foutmelding)

Re: LAN_PLUGIN_NEWS_NAME
erje, vr feb. 19 2021, 10:24A.M.

Als je wisselt van Standaardsjabloon van lijst naar standaard is dat er niet, denk dat ze daar ergens iets niet goed hebben gedaan..
Re: LAN_PLUGIN_NEWS_NAME
Leotgtje, vr feb. 19 2021, 10:59A.M.

Goeie Erje!! . lijkt wel dat men wel PAGE_NAME heeft verandert naar LAN_PLUGIN_NEWS_NAME (zijnde een breadcrumb) zie > https://github.com/e107inc/e107/commit/ba6344bb281c0c8562871cc8a3569ae3015543a3
news .php.
Echter wat Alex al meldde er is geen LAn (define).. ik ga er eens mee aan de slag, en wat stoeien .
Re: LAN_PLUGIN_NEWS_NAME
erje, vr feb. 19 2021, 11:15A.M.

Die lan define is er volgens mij nooit geweest ook niet voor wat er al was. Wat weg is gehaald was niet meer in gebruik maar daar wringt denk ik de schoen, er zijn wel dingen aangepast maar heeft niet gebracht wat het zou moeten moeten.
Re: LAN_PLUGIN_NEWS_NAME
Leotgtje, vr feb. 19 2021, 12:25P.M.

Correct Erje, aangezien ik test op php 8 heb ik een even gekeken wat news op dat systeem doet.. en jawel hoor (ik plak even de uitkomst) 

Fatal error: Uncaught Error: Undefined constant "LAN_PLUGIN_NEWS_NAME" in e107e107_pluginsnewsnews.php:86 Stack trace: #0 e107e107_pluginsnewsnews.php(75): news_front->setBreadcrumb() #1 e107e107_pluginsnewsnews.php(1890): news_front->__construct() #2 e107news.php(23): require_once('htdo...') #3 {main} thrown in e107e107_pluginsnewsnews.php on line 86

sorry voor de backslashes, die worden gestript.

Er moet dus een define bij ( de LAN bestaat maar eindigt nergens)

Ok, gevonden : die moet nog in taal/lan_news php terecht komen.
Heb inmiddels een commit doorgevoerd naar E107 github.
Dank je wel Erje

Re: LAN_PLUGIN_NEWS_NAME
Leotgtje, vr feb. 19 2021, 12:26P.M.

ps Ik wacht even op goedkeuring, dan gaat hij NL in.
Re: LAN_PLUGIN_NEWS_NAME
Alex, vr feb. 19 2021, 01:51P.M.

Toppie!
Re: LAN_PLUGIN_NEWS_NAME
erje, vr feb. 19 2021, 05:04P.M.

En klaar alweer...
Re: LAN_PLUGIN_NEWS_NAME
Leotgtje, vr feb. 19 2021, 06:37P.M.

Yep, en hij zit in het nl pack. Werd een global file van gemaakt met define..
Re: LAN_PLUGIN_NEWS_NAME
erje, vr feb. 19 2021, 07:06P.M.

Haal jij ook overbodige dingen uit de taal bestanden of werk je met de ingebouwde optie in e107 zelf.
Die laat vaak dingen staan die uit het Engelse pakket zijn verwijderd. Heb ze laatst allemaal stuk voor stuk naast elkaar dus eng naast dutch met notepad++ bekeken en ze dan maar gelijk gemaakt , uiteraard niet wat taal betreft maar wat er wel en niet meer instond.
Re: LAN_PLUGIN_NEWS_NAME
Leotgtje, za feb. 20 2021, 04:40P.M.

Als je mij bedoeld, meestal doe ik dat (niet altijd) wel voor de lang pack voor e107 versies.
Voor de bestanden die ik en Alex gebruiken op SF niet zo zeer. Het kan wel, maar toch is het wel eens handig (als je tegen oude installs aanloopt) eens iets terug te vinden.
Ik gebruik inderdaad wel eens de ingebouwde controle, die wijst mij snel op dingetjes (zijn er nog niet of foutjes...) maar leidraad is de github ontwikkeling, die proberen wij bij te houden.
Ja en dan krijg je dus eigenlijk bestanden die extra 'last' in data hebben (vergroot helaas ook foutkansen), maar als je direct overgaat om al de oude dingen eruit te halen (je moet eigenlijk kunnen vertrouwen op de git build) dat die in orde is, komt het dan toch wel eens voor dat een verouderde define resultaat geeft bijv. wanneer een verandering niet netjes is doorgevoerd..
Het is wikken en wegen.