search

victorjonsson / Arlima

Article List Manager - Wordpress plugin suitable for online newspapers that's in need of a fully customizable front page
28 stars 16 forks source link

Video instead of photo for teaser causes fatal error #48

Open benjitavos opened 10 years ago

benjitavos commented 10 years ago

On Arlima Manage list tab, after selecting an article in a list, the user can browse for a teaser photo from the available media library. The problem is that in media library the user can also select a video, and after adding a video file for a teaser the page that is supposed to render the Arlima list is not rendered correctly. All posts up to the post with the video file is rendered, but the rest is failing to render correctly, a fatal error is thrown.

Arlima version deployed: 3.0 beta 34

victorjonsson commented 10 years ago

Videos are not supported at the moment. The first simple step we need to take is to only display images in the media library pop-up (if possible). Later on we can discus whether or not this feature (the rendering of videos in arlima articles) should be implemented.

benjitavos commented 10 years ago

Filtering out videos from the media library pop-up would be a great start, this way we can at least avoid errors if the user is not aware of the restrictions.

victorjonsson commented 10 years ago

yes, it should be a quick fix

benjitavos commented 10 years ago

Will you be able to perform this fix in near future @victorjonsson?

victorjonsson commented 10 years ago

Didn't find any easy way to filter out images in the media library but now the user will get an alert saying that only images can be added to the article.

benjitavos commented 10 years ago

Thanks, that fix is good enough for us at this point. But if the other solution we discussed earlier is possible to implement later on it would be even better.