Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

frontend preview for news articles #7504

Closed
wants to merge 2 commits into from
Closed

frontend preview for news articles #7504

wants to merge 2 commits into from

Conversation

wtom
Copy link

@wtom wtom commented Dec 8, 2014

just a little feature to use the frontend preview on news articles

@@ -288,7 +288,7 @@ protected function getMetaFields($objArticle)
* @param boolean
* @return string
*/
protected function generateNewsUrl($objItem, $blnAddArchive=false)
public function generateNewsUrl($objItem, $blnAddArchive=false)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be public static function otherwise it will trigger a "non-static function cannot be called statically" error.

@Toflar
Copy link
Member

Toflar commented Mar 23, 2015

That would be cool for events as well but the code cannot work, as the generateNewsUrl() currently also has some object references in it (e.g. $this->generateFrontendUrl()). Also, it should be moved to News instead of residing on ModuleNews.

@leofeyer leofeyer modified the milestones: 3.5.0, 4.1.0 Apr 30, 2015
@jankout
Copy link

jankout commented Aug 13, 2015

+1

leofeyer added a commit to contao/news-bundle that referenced this pull request Oct 26, 2015
leofeyer added a commit to contao/calendar-bundle that referenced this pull request Oct 26, 2015
@leofeyer
Copy link
Member

Implemented in contao/core-bundle@43797a4, contao/calendar-bundle@5b4f705 and contao/news-bundle@6e83434.

@leofeyer leofeyer closed this Oct 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants