You are viewing a plain text version of this content. The canonical link for it is here.
Posted to photark-dev@incubator.apache.org by "Avdhesh Yadav (JIRA)" <ji...@apache.org> on 2010/02/05 17:27:27 UTC

[jira] Created: (PHOTARK-12) Image content should be saved and read from in JCR Repository.

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


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.


[jira] Commented: (PHOTARK-12) Image content should be saved and read from in JCR Repository.

Posted by "Luciano Resende (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/PHOTARK-12?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12833072#action_12833072 ] 

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

Updated patch seems ok, please go ahead and commit it.

> 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, imagedisplayer_improved.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.


[jira] Updated: (PHOTARK-12) Image content should be saved and read from in JCR Repository.

Posted by "Avdhesh Yadav (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PHOTARK-12?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Avdhesh Yadav updated PHOTARK-12:
---------------------------------

    Attachment: imagedisplayer_improved.patch

this is improved patch...Please review it.

> 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, imagedisplayer_improved.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.


[jira] Resolved: (PHOTARK-12) Image content should be saved and read from in JCR Repository.

Posted by "Luciano Resende (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PHOTARK-12?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Luciano Resende resolved PHOTARK-12.
------------------------------------

    Resolution: Fixed

Fixed. JCR support is now enhanced and images are directly being stored and served from the repository.

> 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, imagedisplayer_improved.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.


[jira] Commented: (PHOTARK-12) Image content should be saved and read from in JCR Repository.

Posted by "Luciano Resende (JIRA)" <ji...@apache.org>.
    [ 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.


[jira] Updated: (PHOTARK-12) Image content should be saved and read from in JCR Repository.

Posted by "Avdhesh Yadav (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PHOTARK-12?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Avdhesh Yadav updated PHOTARK-12:
---------------------------------

    Attachment: imagedisplayer.patch

Providing patch for quick review.This may not be the final patch for this issue....

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