[magnolia-dev] [JIRA] Issue Comment Edited: (MGNLSTK-623) Review all places where links are generated in STK

2010-06-03 Thread JIRA (on behalf of Jan Haderka)


[ 
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/3/10 4:11 PM:
-

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:red}variation.createLink => VariationImpl, 
ImageOperationProvidingVariation .createLink() is not site aware!{color}
{color:red}mgnl.createLink => LinkUtil.createAbsoluteLink(){color}

flash.model.link => stk.getAssetLink

calendar.item.link => model.eventLink || model.dateEventLink
{color:red}calendar.model.getEventLink => LinkUtil.createAbsoluteLink(){color}
{color:red}calendar.model.getDateEventLink => 
LinkUtil.createAbsoluteLink(){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.

  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:red}variation.createLink => VariationImpl, 
ImageOperationProvidingVariation .createLink() is not site aware!{color}
{color:red}mgnl.createLink => LinkUtil.createAbsoluteLink(){color}

flash.model.link => stk.getAssetLink

calendar.item.link => model.eventLink || model.dateEventLink
{color:red}calendar.model.getEventLink => LinkUtil.createAbsoluteLink(){color}
{color:red}calendar.model.getDateEventLink => 
LinkUtil.createAbsoluteLink(){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 =>

[magnolia-dev] [JIRA] Issue Comment Edited: (MGNLSTK-623) Review all places where links are generated in STK

2010-06-04 Thread JIRA (on behalf of Jan Haderka)


[ 
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 8:57 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:red}variation.createLink => VariationImpl, 
ImageOperationProvidingVariation .createLink() is not site aware!{color}
{color:red}mgnl.createLink => LinkUtil.createAbsoluteLink(){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.

  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:red}variation.createLink => VariationImpl, 
ImageOperationProvidingVariation .createLink() is not site aware!{color}
{color:red}mgnl.createLink => LinkUtil.createAbsoluteLink(){color}

flash.model.link => stk.getAssetLink

calendar.item.link => model.eventLink || model.dateEventLink
{color:red}calendar.model.getEventLink => LinkUtil.createAbsoluteLink(){color}
{color:red}calendar.model.getDateEventLink => 
LinkUtil.createAbsoluteLink(){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 wit

[magnolia-dev] [JIRA] Issue Comment Edited: (MGNLSTK-623) Review all places where links are generated in STK

2010-06-04 Thread JIRA (on behalf of Jan Haderka)


[ 
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:24 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.
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.

  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:red}variation.createLink => VariationImpl, 
ImageOperationProvidingVariation .createLink() is not site aware!{color}
{color:red}mgnl.createLink => LinkUtil.createAbsoluteLink(){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

[magnolia-dev] [JIRA] Issue Comment Edited: (MGNLSTK-623) Review all places where links are generated in STK

2010-06-04 Thread JIRA (on behalf of Jan Haderka)


[ 
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.lin