search

RSS-Bridge / rss-bridge

The RSS feed for websites missing it
https://rss-bridge.org/bridge01/
The Unlicense
6.99k stars 1.02k forks source link

[PepperBridgeAbstract,DealabsBridge,HotUKDealsBridge,MydealsBridge] Move as much as possible to JSON #4032

Closed sysadminstory closed 3 months ago

sysadminstory commented 3 months ago

As the website use more and more JSON, and JSON is a machine readable format, I migrated as much as possible to the JSON.

This simplifies the Abstract class a lot, and the Bridge classes need less language specific strings.

github-actions[bot] commented 3 months ago

Pull request artifacts

Bridge Context Status
Dealabs 1 Recherche par Mot(s) clé(s) (current) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 105
⚠️ DEBUG lib/FeedItem.php(177): Author must be a string!
⚠️ WARNING Cannot modify header information - headers already sent by (output started at lib/logger.php:154) at lib/http.php line 346
Dealabs 1 Recherche par Mot(s) clé(s) (pr) ✔️
Dealabs 2 Deals par groupe (current) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 105
⚠️ DEBUG lib/FeedItem.php(177): Author must be a string!
⚠️ WARNING Cannot modify header information - headers already sent by (output started at lib/logger.php:154) at lib/http.php line 346
Dealabs 2 Deals par groupe (pr) ✔️
Dealabs 3 Surveillance Discussion (current) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 313
Dealabs 3 Surveillance Discussion (pr) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 267
HotUKDeals 1 Search by keyword(s)) (current) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 105
⚠️ DEBUG lib/FeedItem.php(177): Author must be a string!
⚠️ WARNING Cannot modify header information - headers already sent by (output started at lib/logger.php:154) at lib/http.php line 346
HotUKDeals 1 Search by keyword(s)) (pr) ✔️
HotUKDeals 2 Deals per group (current) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 105
⚠️ DEBUG lib/FeedItem.php(177): Author must be a string!
⚠️ WARNING Cannot modify header information - headers already sent by (output started at lib/logger.php:154) at lib/http.php line 346
HotUKDeals 2 Deals per group (pr) ✔️
HotUKDeals 3 Discussion Monitoring (current) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 313
HotUKDeals 3 Discussion Monitoring (pr) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 267
Mydeals 1 Suche nach Stichworten (current) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 105
⚠️ DEBUG lib/FeedItem.php(177): Author must be a string!
⚠️ WARNING Cannot modify header information - headers already sent by (output started at lib/logger.php:154) at lib/http.php line 346
Mydeals 1 Suche nach Stichworten (pr) ✔️
Mydeals 2 Deals pro Gruppen (current) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 105
⚠️ DEBUG lib/FeedItem.php(177): Author must be a string!
⚠️ WARNING Cannot modify header information - headers already sent by (output started at lib/logger.php:154) at lib/http.php line 346
Mydeals 2 Deals pro Gruppen (pr) ✔️
Mydeals 3 Überwachung Diskussion (current) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 313
Mydeals 3 Überwachung Diskussion (pr) ⚠️ WARNING Attempt to read property "plaintext" on null at bridges/PepperBridgeAbstract.php line 267

last change: Tuesday 2024-03-26 23:13:08

dvikan commented 3 months ago

you maybe want to Json::decode($data, true) to get a \stdClass instead of array back.

Also some new errors which will error in later php 8 versions.