[ 
https://issues.apache.org/jira/browse/PHOTARK-12?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12830410#action_12830410
 ] 

Luciano Resende commented on PHOTARK-12:
----------------------------------------

Overall, is on the right direction... 

Minor comment is around the image location below... 
-           img.src = "gallery/" + albumName + "/" + albumItems[albumPos];
+           img.src = window.location.href + "ImageDisplayer/"+ this.albumName 
+"/" + albumItems[albumPos];

I think we can possible have two options
   - follow a rest style, and allow the image to be accessed via 
gallery/album/image
   - or let the server to provide the image location via a image property


> Image content should be saved and read from in JCR Repository.
> --------------------------------------------------------------
>
>                 Key: PHOTARK-12
>                 URL: https://issues.apache.org/jira/browse/PHOTARK-12
>             Project: PhotArk
>          Issue Type: Improvement
>          Components: PhotArk WebApp
>    Affects Versions: PhotArk M2
>            Reporter: Avdhesh Yadav
>             Fix For: PhotArk M2
>
>         Attachments: imagedisplayer.patch
>
>
> Image content should be saved in JCR Repository and read from JCR 
> repository.There is no need to save images in filesystem.We have JCR 
> Repository which should be used for saving images.Image contents read from 
> JCR repository using tuscany.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to