[
http://jira.magnolia-cms.com/browse/MGNLSTK-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=28562#action_28562
]
Jan Haderka edited comment on MGNLSTK-623 at 6/4/10 9:26 AM:
-------------------------------------------------------------
Reviewed all templates and associated models. Reviewed items listed below.
{panel}
stk.getAssetLink() => mgnl.createLink() || imaging.createLink()
stk.getImageLink() => mgnl.createLink() || imaging.createLink()
stk.getAssetVariation.link => variation.createLink()
imaging.createLink => mgnl.createLink() || variation.createLink()
{color:blue}variation.createLink => VariationImpl,
ImageOperationProvidingVariation .createLink(){color}
{color:blue}mgnl.createLink => LinkUtil.createLink(){color}
flash.model.link => stk.getAssetLink
calendar.item.link => model.eventLink || model.dateEventLink
{color:blue}calendar.model.getEventLink => -LinkUtil.createAbsoluteLink()-
mgnl.createLink(){color}
{color:blue}calendar.model.getDateEventLink => -LinkUtil.createAbsoluteLink()-
mgnl.createLink(){color}
eventOverview.model.prevLink => eventOverview.model.link.href
eventOverview.model.nextLink => eventOverview.model.link.href
{color:red}eventOverview.model.link.href => mgnl.createLink() !! expects links
to end with .html!!{color}
searchResult.item.link => mgnl.createLink()
searchResult.model.searchPageLink => STKTemplateModel.getSearchPageLink()
STKTemplateModel.getSearchPageLink() => mgnl.createLink()
image.link=>STK.getAsset().getLink()
downloadLink.model.link => mgnl.createLink()
externalLink.model.link => STKUtil.getExternalLink()
internalLink.model.link => mgnl.createLink()
pagination.pager.getPageLink() => parses current page link
AbstractItemListModel.pager => mgnl.createLink
{color:red}CategoryOverviewModel.pager => manually combine link from context
path and currentURI{color}
stageXL.model.teaserLink => AbstractTeaserModel.teaserLink
AbstractTeaserModel.teaserLink => mgnl.createLink()
stageXL.model.image.link => ImageModel.image => stk.getImageLink()
carouselItem.teaserLink => AbstractTeaserModel.teaserLink
downloadFile.model.teaserLink => AbstractTeaserModel.teaserLink
eventList.model.allEventsLink => mgnl.createLink()
externalPage.model.teaserLink => stk.externalLink()
fingerTabbedItem.model.teaserLink => AbstractTeaserModel.teaserLink
internalPage.model.teaserLink => AbstractTeaserModel.teaserLink
newsList.model.allNewsLink => mgnl.createLink()
stkTeaserSingleFeed.item.link => link from the feed
{color:blue}syndicate.model.link=>manually assemble virtual URI, which is OK
since the Virtual URI will be the same no matter which site and domain is
used.{color}
infoblock.model.externalLink => mgnl.createLink()
branding.model.logoImageLink => stk.assetLink
branding.model.printLogoImageLink => stk.assetLink
branding.model.searchPageLink => mgnl.createLink()
branding.model.homeLink => mgnl.createLink()
htmlHeader.cssFile.link => Resource.getLink()
htmlHeader.jsFile.link => Resource.getLink()
{color:red}Resource.getLink() => manually combine context path with
stk-resources prefix with site name and link{color}
mainAreaIntro.model.getCategoryLink => CategorizationSupport.categoryLink
CategorizationSupport.categoryLink => CategoryUtil.categoryLink
{color:red}CategoryUtil.categoryLink => manualy combine context path and handle
to category with name{color}
mainAreaIntro.imageModel.image.link => imaging.createLink()
@cms.links => manually assembles path from context and path to the script
{panel}
Items in read are either questionable or wrong.
was (Author: had):
Reviewed all templates and associated models. Reviewed items listed below.
{panel}
stk.getAssetLink() => mgnl.createLink() || imaging.createLink()
stk.getImageLink() => mgnl.createLink() || imaging.createLink()
stk.getAssetVariation.link => variation.createLink()
imaging.createLink => mgnl.createLink() || variation.createLink()
{color:blue}variation.createLink => VariationImpl,
ImageOperationProvidingVariation .createLink(){color}
{color:blue}mgnl.createLink => LinkUtil.createLink(){color}
flash.model.link => stk.getAssetLink
calendar.item.link => model.eventLink || model.dateEventLink
{color:blue}calendar.model.getEventLink => -LinkUtil.createAbsoluteLink()-
mgnl.createLink(){color}
{color:blue}calendar.model.getDateEventLink => -LinkUtil.createAbsoluteLink()-
mgnl.createLink(){color}
eventOverview.model.prevLink => eventOverview.model.link.href
eventOverview.model.nextLink => eventOverview.model.link.href
{color:red}eventOverview.model.link.href => mgnl.createLink() !! expects links
to end with .html!!{color}
searchResult.item.link => mgnl.createLink()
searchResult.model.searchPageLink => STKTemplateModel.getSearchPageLink()
STKTemplateModel.getSearchPageLink() => mgnl.createLink()
image.link=>STK.getAsset().getLink()
downloadLink.model.link => mgnl.createLink()
externalLink.model.link => STKUtil.getExternalLink()
internalLink.model.link => mgnl.createLink()
pagination.pager.getPageLink() => parses current page link
AbstractItemListModel.pager => mgnl.createLink
{color:red}CategoryOverviewModel.pager => manually combine link from context
path and currentURI{color}
stageXL.model.teaserLink => AbstractTeaserModel.teaserLink
AbstractTeaserModel.teaserLink => mgnl.createLink()
stageXL.model.image.link => ImageModel.image => stk.getImageLink()
carouselItem.teaserLink => AbstractTeaserModel.teaserLink
downloadFile.model.teaserLink => AbstractTeaserModel.teaserLink
eventList.model.allEventsLink => mgnl.createLink()
externalPage.model.teaserLink => stk.externalLink()
fingerTabbedItem.model.teaserLink => AbstractTeaserModel.teaserLink
internalPage.model.teaserLink => AbstractTeaserModel.teaserLink
newsList.model.allNewsLink => mgnl.createLink()
stkTeaserSingleFeed.item.link => link from the feed
{color:blue}syndicate.model.link=>manually assemble virtual URI, which is OK
since the Virtual URI will be the same no matter which site and domain is
used.{color}
infoblock.model.externalLink => mgnl.createLink()
branding.model.logoImageLink => stk.assetLink
branding.model.printLogoImageLink => stk.assetLink
branding.model.searchPageLink => mgnl.createLink()
branding.model.homeLink => mgnl.createLink()
htmlHeader.cssFile.link => Resource.getLink()
htmlHeader.jsFile.link => Resource.getLink()
{color:red}Resource.getLink() => manually combine context path with
stk-resources prefix with site name and link{color}
mainAreaIntro.model.getCategoryLink => CategorizationSupport.categoryLink
CategorizationSupport.categoryLink => CategoryUtil.categoryLink
{color:red}CategoryUtil.categoryLink => manualy combine context path and handle
to category with name{color}
mainAreaIntro.imageModel.image.link => imaging.createLink()
@cms.links => manually assembles path from context and path to the script
{panel}
Items in read are either questionable or wrong.
In general most of the links (those in black) end up using
{{MagnoliaTemplatingUtils.createLink}} method that will create absolute link
which might not be correct in the multisite setup. However those can be
replaced by changing the output of {{LinkTransformerManager.getAbsolute()}} and
therefore can be customized to suit the special setups.
> Review all places where links are generated in STK
> --------------------------------------------------
>
> Key: MGNLSTK-623
> URL: http://jira.magnolia-cms.com/browse/MGNLSTK-623
> Project: Magnolia Standard Templating Kit
> Issue Type: Task
> Components: base system, paragraphs
> Affects Versions: 1.3
> Reporter: Jan Haderka
> Assignee: Philipp Bärfuss
> Fix For: 1.3.x
>
>
> Quite few models in STK generate links to content directly instead of using
> {{LinkUtil}}/{{LinkManager}}. This is then cause of issues when URLs are
> altered by system (e.g. when using multisite support).
> All the links in models should be generated only using appropriate
> {{LinkUtil}} methods.
> This is specially true for overview paragraphs such as {{NewsOverview}} and
> {{EventsOverview}}, but also for pagination ( {{pager.ftl}} macro and related
> {{pager}} objects).
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com>
----------------------------------------------------------------
