-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Update nordbayern #2169
Update nordbayern #2169
Conversation
bridges/NordbayernBridge.php
Outdated
private function fixLinksInElement($element) { | ||
foreach($element->find('a') as $a) { | ||
if(self::startsWith($a->href, '/')) { | ||
$a->href = 'https://nordbayern.de' . $a->href; | ||
} | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use defaultLinkTo
instead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
bridges/NordbayernBridge.php
Outdated
'Rothenburg o.d.T.' => 'rothenburg-o-d-t', | ||
'Schwabach' => 'schwabach', | ||
'Rothenburg o.d.T.' => 'rothenburg-ob-der-tauber', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Old feed urls with these params will become invalid if following change is accepted. Can we escape it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Also please fix phpcs errors |
find articles get raw articles include images again fix serveral bugs catch all articles fix links try to exlucde polizeimeldungen try to fix weird html fix ident make code php7 compatible apply requested changes fix phpcs error
3f8d732
to
6d08d54
Compare
bump 😅 |
gj @theScrabi |
Nordbayern changed the whole layout and design. This is the new adaptation.