Error: Invalid URL

  • Resolved luxurytraveldiary

    (@luxurytraveldiary)


    8 years, 1 month ago

    Hi, Great Plugin. Having an issue with the err “Error: Invalid URL”, regarding posts with relative href’s. Appreciate that Apple News is not keen on links, so my question is shouldn’t these be dealt with by the plugin, (parse_hyperlink_node()), or are the URLs in question breaking something.

    Before I start hacking about though I’d ask in-case I’m missing something – Apple News is new to me.

    {"version":"1.1","identifier":"post-26770","language":"en","title":"British Airways M&S Food Menu On Short Haul Flights","documentStyle":{"backgroundColor":"#FAFAFA"},"layout":{"columns":7,"width":1024,"margin":100,"gutter":20},"components":[{"role":"header","layout":"headerPhotoLayout","components":[{"role":"photo","layout":"headerPhotoLayout","URL":"https:\/\/www.luxurytraveldiary.com\/wp-content\/uploads\/2016\/09\/british-airways-MS-food-menu-short-haul-flights.jpg"}],"behavior":{"type":"parallax","factor":0.8}},{"role":"container","layout":{"columnStart":0,"columnSpan":7,"ignoreDocumentMargin":true},"style":{"backgroundColor":"#FAFAFA"},"components":[{"role":"title","text":"British Airways M&S Food Menu On Short Haul Flights","textStyle":"default-title","layout":"title-layout"},{"role":"byline","text":"by Yasmin | Sep 29, 2016 | 4:15 PM","textStyle":"default-byline","layout":"byline-layout"},{"role":"body","text":"Following the news that British Airways will charge for food on short haul flights from London Heathrow and London Gatwick from January 2017, I wanted to share some of the items that are available on the M&S food menu on short haul flights with their prices.\n\nBreakfast will include Greek style natural yogurt with summer berry compote and granola (\u00a31.95), classic fruit salad with pineapple, melon, mango, apple, kiwi and blueberries (\u00a33.10). \n\nHot breakfast will include the M&S Caf\u00e9 bacon roll with pork from British farmers (\u00a34.75) and a tomato and mozzarella focaccia (\u00a34.75).\n\nSandwiches on offer will include the Aberdeen Angus beef and red onion chutney bloomer (\u00a34.75) and the classic cheese ploughman\u2019s (\u00a33.00) along with a buffalo mozzarella & tomato focaccia (4.75) and a balanced for you spiced chicken with rice & quinoa salad (\u00a34.95).\n\nSnacks will include a nut assortment of almonds, brazil, cashew and hazelnuts (\u00a31.60), wasabi peas (\u00a31.60), salted cashews (\u00a31.60), Oriental snack mix (\u00a32), super fruit, nut and seed flapjacks (\u00a31.45), salt & vinegar and lightly salted hand-cut crisps (both \u00a31), a salted caramel hazelnut millionaire bar (\u00a31.45), a grab-bag sized milk chocolate covered popping popcorn and pretzels (\u00a32.45), packs of mini oat biscuits (\u00a31.10), Swiss milk chocolate mountain bars (\u00a31.70), the famous Percy Pigs (\u00a31.80) and \u2018The Whippy One\u2019 (\u00a31).\n\nDrinks will also be charged for and expect to pay \u00a31.80 for a 330ml Diet Coke, \u00a31.50 for a 150ml Schweppes Tonic Water 150ml, \u00a34.00 for 330ml of London Pride, \u00a34.50 for 5cl of Bombay Sapphire, \u00a31.80 for Water, \u00a32.30 for a cup of English breakfast tea or Java republic coffee, \u00a36 for a spirit & mixer (G&T) and \u00a34.50 for wine.\n\nFree food seems an inevitable loss when travelling in economy on short haul flights, as each airline tries to further reduce their headline flight fares to beat the competition. I have already written about my disappointment in the current [food offering in euro traveller](\/2016\/05\/british-airways-stopping-food-eurotraveller\/) with just a snack and a drink. The fact that British Airways will now charge for food will further encourage me to spend my Avios on Club Europe seats to enjoy priority luggage, more spacious seating and a nice salad or the [BA afternoon tea](\/2016\/09\/british-airways-club-europe-afternoon-tea\/). The lack of food in Euro Traveller makes it more [worthwhile spending you Avios on upgrading to Club Europe](\/2016\/04\/worth-spending-avios-british-airways-club-europe\/). Of course you could instead fly Ryanair or Easyjet who have similarly sparse offerings.\n\nIf you enjoyed this article you may also want to take a first look at [Heathrow terminal 6](\/2016\/09\/picture-review-london-heathrow-terminal-6\/) and enjoy my [airline reviews](\/category\/transport\/airlines\/) and my experiences in [British Airways First](\/2016\/04\/review-british-airways-dreamliner-b787-9-first-class-london-muscat\/) and [Business Class on the B777](\/2016\/08\/british-airways-b777-300er-club-world-review-tokyo-to-london\/) and the [B747 Jumbo](\/2016\/08\/british-airways-b747-jumbo-jet-club-world-review\/).\n\n[British Airways](http:\/\/prf.hn\/click\/camref:111l5TW)","format":"markdown","textStyle":"dropcapBodyStyle","layout":"body-layout-last"}]}],"componentTextStyles":{"dropcapBodyStyle":{"textAlignment":"left","fontName":"AvenirNext-Regular","fontSize":18,"lineHeight":24,"textColor":"#4f4f4f","linkStyle":{"textColor":"#428BCA"},"paragraphSpacingBefore":18,"paragraphSpacingAfter":18,"dropCapStyle":{"numberOfLines":4,"numberOfCharacters":1,"padding":5,"fontName":"AvenirNext-Bold","textColor":"#4f4f4f"}},"default-body":{"textAlignment":"left","fontName":"AvenirNext-Regular","fontSize":18,"lineHeight":24,"textColor":"#4f4f4f","linkStyle":{"textColor":"#428BCA"},"paragraphSpacingBefore":18,"paragraphSpacingAfter":18},"default-title":{"fontName":"AvenirNext-Bold","fontSize":48,"lineHeight":52,"textColor":"#333333","textAlignment":"left"},"default-byline":{"textAlignment":"left","fontName":"AvenirNext-Medium","fontSize":13,"textColor":"#7c7c7c"}},"componentLayouts":{"body-layout":{"columnStart":0,"columnSpan":5,"margin":{"top":12,"bottom":12}},"body-layout-last":{"columnStart":0,"columnSpan":5,"margin":{"top":12,"bottom":30}},"headerPhotoLayout":{"columnStart":0,"columnSpan":7,"ignoreDocumentMargin":true},"headerBelowTextPhotoLayout":{"columnStart":0,"columnSpan":7,"ignoreDocumentMargin":true,"margin":{"top":30,"bottom":0}},"title-layout":{"margin":{"top":30,"bottom":0}},"byline-layout":{"columnStart":0,"columnSpan":7,"margin":{"top":10,"bottom":10}}},"metadata":{"excerpt":"Following the news that British Airways will charge for food on short haul flights from London Heathrow and London Gatwick from January 2017, I wanted to share some of the items that are available on the M&S food menu on short haul flights with their prices. Breakfast will include Greek style natural yogurt with summer...","thumbnailURL":"https:\/\/www.luxurytraveldiary.com\/wp-content\/uploads\/2016\/09\/british-airways-MS-food-menu-short-haul-flights.jpg","dateCreated":"2016-09-29T15:15:21+01:00","dateModified":"2016-09-29T15:27:01+01:00","datePublished":"2016-09-29T15:15:21+01:00","canonicalURL":"https:\/\/www.luxurytraveldiary.com\/2016\/09\/british-airways-food-menu-on-short-haul-flights\/","generatorIdentifier":"publish-to-apple-news","generatorName":"Publish to Apple News","generatorVersion":"1.1.8"},"advertisingSettings":{"frequency":1,"layout":{"margin":{"top":15,"bottom":15}}}}

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Bradford Campeau-Laurion

    (@potatomaster)

    This is already a known issue, slated for resolution in a future plugin version: https://github.com/alleyinteractive/apple-news/issues/148

    Thread Starter luxurytraveldiary

    (@luxurytraveldiary)

    Yup, saw that. Sorry – meant to respond to my own post. Wrote a quick and dirty patch to tide us over.

    diff --git a/class-markdown.php b/class-markdown.php
index 18d17af..0e0c49d 100644
--- a/class-markdown.php
+++ b/class-markdown.php
@@ -184,9 +184,11 @@ class Markdown {
 	 * @access private
 	 */
 	private function parse_hyperlink_node( $node ) {
+		$url = apply_filters( 'apple_news_markdown_hyperlink', $node->getAttribute( 'href' ) );
 
+		if ( preg_match( '/^(?!www\.|(?:http|ftp)s?:\/\/|[A-Za-z]:\\\\|\/\/).*/m',  $url ) ) $url = site_url() . $url;
+
 		return '[' . $this->parse_nodes( $node->childNodes ) . '](' . esc_url_raw( $url ) . ')';
 	}
--
    Plugin Author Bradford Campeau-Laurion

    (@potatomaster)

    Please note if you patch the core plugin code, you’ll be unable to receive future automatic plugin updates.

    Feel free to submit any code to the plugin as a pull request since it’s open source: https://github.com/alleyinteractive/apple-news.

    Thread Starter luxurytraveldiary

    (@luxurytraveldiary)

    Aware. Just a quick fix to get our posts to play with this rel. Would hate to submit until the regex is properly tested and I’ve made myself a little more familiar with your source. Will do so this wend.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Error: Invalid URL’ is closed to new replies.