cleverdevil / Known-Food

Food and drink logging for Known
Other
5 stars 4 forks source link

Problem on permalink when attempting to POSSE #1

Open chrisaldrich opened 7 years ago

chrisaldrich commented 7 years ago

First! But seriously... This looks great!

I've installed on Known 0.9.2 and tried two test posts: http://stream.boffosocko.com/2017/cheese-danish http://stream.boffosocko.com/2017/venti-caramel-macchiato

The cheese danish worked as expected, but I tried to POSSE the caramel macchiato to Twitter and encountered a problem. The post shows up properly in my primary stream, but it didn't POSSE to Twitter and then when clicking on the permalink for the post I get the following error:

Oh no! Known experienced a problem! Known experienced a problem with this page and couldn't continue. The technical details are as follows:

`Fatal Error: /homepages/32/d491429024/htdocs/idno/Idno/Common/Page.php:116 - "Uncaught TypeError: Argument 1 passed to Idno\Common\Page::exception() must be an instance of Exception, instance of TypeError given, called in /Idno/Common/Page.php on line 71 and defined in /Idno/Common/Page.php:116 Stack trace:

0 /Idno/Common/Page.php(71): Idno\Common\Page->exception(Object(TypeError))

1 [internal function]: Idno\Common\Page->Idno\Common{closure}(Object(TypeError))

2 {main}

thrown", on page stream.boffosocko.com/2017/venti-caramel-macchiato`

cleverdevil commented 7 years ago

I think I've fixed this in the latest commit. Can you check if it resolves your issue?

chrisaldrich commented 7 years ago

Jonathan, perfect, the new version fixes the issues with the content showing on the permalink.

The separate, but somewhat related issue is that the post interface still shows all of the UI for syndicating to all of the options (facebook, twitter, WordPress, Tumblr, LinkedIn, etc.), yet choosing those options doesn't actually syndicate the content. (Typically I wouldn't POSSE these post types anywhere anyway, but other users may expect this piece to work if it's shown.) I know some post types in Known hide the syndication targets when they're not supported (ie, only audio related posts can be syndicated to soundcloud), but I'm not sure how these are done.

Interestingly, when editing these types of posts, the syndication targets (correctly) aren't shown in the editable version.