You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by "David Johnson (JIRA)" <no...@atlassian.com> on 2007/05/07 22:09:55 UTC

[Roller-JIRA] Created: (ROL-1406) Optional themes use incorrect $url method

Optional themes use incorrect $url method
-----------------------------------------

                 Key: ROL-1406
                 URL: http://opensource.atlassian.com/projects/roller/browse/ROL-1406
             Project: Roller
          Issue Type: Bug
          Components: Theme specific
    Affects Versions: 3.1
            Reporter: David Johnson
            Assignee: Roller Unassigned


>From Sedat:
> In roller 3.1 oprional themes, $url.resource(String
> file)  macro is used to get the resources
> (images,styles etc) of a theme. But this will not
> work, and the appearance of theme is not seen
> correctly. I change the usage of $url.resource(String
> file) with $url.themeResource(String theme, String
> file) macro and the themes are working correctly now.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/roller/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[Roller-JIRA] Resolved: (ROL-1406) Optional themes use incorrect $url method

Posted by "David Johnson (JIRA)" <no...@atlassian.com>.
     [ http://opensource.atlassian.com/projects/roller/browse/ROL-1406?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Johnson resolved ROL-1406.
--------------------------------

    Resolution: Fixed

Fixed in 3.1 branch

> Optional themes use incorrect $url method
> -----------------------------------------
>
>                 Key: ROL-1406
>                 URL: http://opensource.atlassian.com/projects/roller/browse/ROL-1406
>             Project: Roller
>          Issue Type: Bug
>          Components: Theme specific
>    Affects Versions: 3.1
>            Reporter: David Johnson
>            Assignee: Roller Unassigned
>             Fix For: 3.1.1
>
>
> From Sedat:
> > In roller 3.1 oprional themes, $url.resource(String
> > file)  macro is used to get the resources
> > (images,styles etc) of a theme. But this will not
> > work, and the appearance of theme is not seen
> > correctly. I change the usage of $url.resource(String
> > file) with $url.themeResource(String theme, String
> > file) macro and the themes are working correctly now.
> Richard Jones wrote:
> > Correct me if I'm wrong but I thought that now in 3.1 we were using
> > $url.resource() as opposed to $url.themeResource()...  because now the
> > resources are copied across when you customize a theme so instead of
> > using the original theme resource, you would use your own copy.
> >
> > Just checked 'sotto' & 'brushmetal' theme (both shipped with roller)
> > and they use $url.resource() and not $url.themeResource().
> Allen G:
> > yes, that is correct and it's *imperative* that you use $url.resource()
> > this way to prevent pointers to shared resources.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/roller/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[Roller-JIRA] Assigned: (ROL-1406) Optional themes use incorrect $url method

Posted by "David Johnson (JIRA)" <no...@atlassian.com>.
     [ http://opensource.atlassian.com/projects/roller/browse/ROL-1406?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Johnson reassigned ROL-1406:
----------------------------------

    Assignee: David Johnson  (was: Roller Unassigned)

> Optional themes use incorrect $url method
> -----------------------------------------
>
>                 Key: ROL-1406
>                 URL: http://opensource.atlassian.com/projects/roller/browse/ROL-1406
>             Project: Roller
>          Issue Type: Bug
>          Components: Theme specific
>    Affects Versions: 3.1
>            Reporter: David Johnson
>            Assignee: David Johnson
>             Fix For: 3.1.1
>
>
> From Sedat:
> > In roller 3.1 oprional themes, $url.resource(String
> > file)  macro is used to get the resources
> > (images,styles etc) of a theme. But this will not
> > work, and the appearance of theme is not seen
> > correctly. I change the usage of $url.resource(String
> > file) with $url.themeResource(String theme, String
> > file) macro and the themes are working correctly now.
> Richard Jones wrote:
> > Correct me if I'm wrong but I thought that now in 3.1 we were using
> > $url.resource() as opposed to $url.themeResource()...  because now the
> > resources are copied across when you customize a theme so instead of
> > using the original theme resource, you would use your own copy.
> >
> > Just checked 'sotto' & 'brushmetal' theme (both shipped with roller)
> > and they use $url.resource() and not $url.themeResource().
> Allen G:
> > yes, that is correct and it's *imperative* that you use $url.resource()
> > this way to prevent pointers to shared resources.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/roller/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[Roller-JIRA] Updated: (ROL-1406) Optional themes use incorrect $url method

Posted by "David Johnson (JIRA)" <no...@atlassian.com>.
     [ http://opensource.atlassian.com/projects/roller/browse/ROL-1406?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Johnson updated ROL-1406:
-------------------------------

    Description: 
>From Sedat:
> In roller 3.1 oprional themes, $url.resource(String
> file)  macro is used to get the resources
> (images,styles etc) of a theme. But this will not
> work, and the appearance of theme is not seen
> correctly. I change the usage of $url.resource(String
> file) with $url.themeResource(String theme, String
> file) macro and the themes are working correctly now.

Richard Jones wrote:
> Correct me if I'm wrong but I thought that now in 3.1 we were using
> $url.resource() as opposed to $url.themeResource()...  because now the
> resources are copied across when you customize a theme so instead of
> using the original theme resource, you would use your own copy.
>
> Just checked 'sotto' & 'brushmetal' theme (both shipped with roller)
> and they use $url.resource() and not $url.themeResource().

Allen G:
> yes, that is correct and it's *imperative* that you use $url.resource()
> this way to prevent pointers to shared resources.

  was:
>From Sedat:
> In roller 3.1 oprional themes, $url.resource(String
> file)  macro is used to get the resources
> (images,styles etc) of a theme. But this will not
> work, and the appearance of theme is not seen
> correctly. I change the usage of $url.resource(String
> file) with $url.themeResource(String theme, String
> file) macro and the themes are working correctly now.


> Optional themes use incorrect $url method
> -----------------------------------------
>
>                 Key: ROL-1406
>                 URL: http://opensource.atlassian.com/projects/roller/browse/ROL-1406
>             Project: Roller
>          Issue Type: Bug
>          Components: Theme specific
>    Affects Versions: 3.1
>            Reporter: David Johnson
>            Assignee: Roller Unassigned
>
> From Sedat:
> > In roller 3.1 oprional themes, $url.resource(String
> > file)  macro is used to get the resources
> > (images,styles etc) of a theme. But this will not
> > work, and the appearance of theme is not seen
> > correctly. I change the usage of $url.resource(String
> > file) with $url.themeResource(String theme, String
> > file) macro and the themes are working correctly now.
> Richard Jones wrote:
> > Correct me if I'm wrong but I thought that now in 3.1 we were using
> > $url.resource() as opposed to $url.themeResource()...  because now the
> > resources are copied across when you customize a theme so instead of
> > using the original theme resource, you would use your own copy.
> >
> > Just checked 'sotto' & 'brushmetal' theme (both shipped with roller)
> > and they use $url.resource() and not $url.themeResource().
> Allen G:
> > yes, that is correct and it's *imperative* that you use $url.resource()
> > this way to prevent pointers to shared resources.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/roller/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[Roller-JIRA] Commented: (ROL-1406) Optional themes use incorrect $url method

Posted by "David Johnson (JIRA)" <no...@atlassian.com>.
    [ http://opensource.atlassian.com/projects/roller/browse/ROL-1406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_13823 ] 

David Johnson commented on ROL-1406:
------------------------------------

See also the thread associated with this email:
http://mail-archives.apache.org/mod_mbox/roller-user/200706.mbox/%3c4678b55c.1238400a.1fcd.5fbb@mx.google.com%3e

> Optional themes use incorrect $url method
> -----------------------------------------
>
>                 Key: ROL-1406
>                 URL: http://opensource.atlassian.com/projects/roller/browse/ROL-1406
>             Project: Roller
>          Issue Type: Bug
>          Components: Theme specific
>    Affects Versions: 3.1
>            Reporter: David Johnson
>            Assignee: Roller Unassigned
>             Fix For: 3.1.1
>
>
> From Sedat:
> > In roller 3.1 oprional themes, $url.resource(String
> > file)  macro is used to get the resources
> > (images,styles etc) of a theme. But this will not
> > work, and the appearance of theme is not seen
> > correctly. I change the usage of $url.resource(String
> > file) with $url.themeResource(String theme, String
> > file) macro and the themes are working correctly now.
> Richard Jones wrote:
> > Correct me if I'm wrong but I thought that now in 3.1 we were using
> > $url.resource() as opposed to $url.themeResource()...  because now the
> > resources are copied across when you customize a theme so instead of
> > using the original theme resource, you would use your own copy.
> >
> > Just checked 'sotto' & 'brushmetal' theme (both shipped with roller)
> > and they use $url.resource() and not $url.themeResource().
> Allen G:
> > yes, that is correct and it's *imperative* that you use $url.resource()
> > this way to prevent pointers to shared resources.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/roller/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[Roller-JIRA] Commented: (ROL-1406) Optional themes use incorrect $url method

Posted by "David Johnson (JIRA)" <no...@atlassian.com>.
    [ http://opensource.atlassian.com/projects/roller/browse/ROL-1406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_13837 ] 

David Johnson commented on ROL-1406:
------------------------------------

Apparently, there was a problem during theme packaging and some of the themes were not the latest from the 3.x branch. So some themes still used the themeResource call. And some used had an incorrect extra 2nd in their $url.resource() calls (sunset and x2).

I tested $url.resource() with shared, preview and customized themes and it worked fine everytime. When customizing a theme, the theme resources are automatically and correctly copied into the weblog's file upload area.

I'll repackage the themes and release give them the version 3.1.1

> Optional themes use incorrect $url method
> -----------------------------------------
>
>                 Key: ROL-1406
>                 URL: http://opensource.atlassian.com/projects/roller/browse/ROL-1406
>             Project: Roller
>          Issue Type: Bug
>          Components: Theme specific
>    Affects Versions: 3.1
>            Reporter: David Johnson
>            Assignee: Roller Unassigned
>             Fix For: 3.1.1
>
>
> From Sedat:
> > In roller 3.1 oprional themes, $url.resource(String
> > file)  macro is used to get the resources
> > (images,styles etc) of a theme. But this will not
> > work, and the appearance of theme is not seen
> > correctly. I change the usage of $url.resource(String
> > file) with $url.themeResource(String theme, String
> > file) macro and the themes are working correctly now.
> Richard Jones wrote:
> > Correct me if I'm wrong but I thought that now in 3.1 we were using
> > $url.resource() as opposed to $url.themeResource()...  because now the
> > resources are copied across when you customize a theme so instead of
> > using the original theme resource, you would use your own copy.
> >
> > Just checked 'sotto' & 'brushmetal' theme (both shipped with roller)
> > and they use $url.resource() and not $url.themeResource().
> Allen G:
> > yes, that is correct and it's *imperative* that you use $url.resource()
> > this way to prevent pointers to shared resources.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/roller/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[Roller-JIRA] Commented: (ROL-1406) Optional themes use incorrect $url method

Posted by "David Johnson (JIRA)" <no...@atlassian.com>.
    [ http://opensource.atlassian.com/projects/roller/browse/ROL-1406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_13851 ] 

David Johnson commented on ROL-1406:
------------------------------------

See also ROL-1471

> Optional themes use incorrect $url method
> -----------------------------------------
>
>                 Key: ROL-1406
>                 URL: http://opensource.atlassian.com/projects/roller/browse/ROL-1406
>             Project: Roller
>          Issue Type: Bug
>          Components: Theme specific
>    Affects Versions: 3.1
>            Reporter: David Johnson
>            Assignee: David Johnson
>             Fix For: 3.1.1
>
>
> From Sedat:
> > In roller 3.1 oprional themes, $url.resource(String
> > file)  macro is used to get the resources
> > (images,styles etc) of a theme. But this will not
> > work, and the appearance of theme is not seen
> > correctly. I change the usage of $url.resource(String
> > file) with $url.themeResource(String theme, String
> > file) macro and the themes are working correctly now.
> Richard Jones wrote:
> > Correct me if I'm wrong but I thought that now in 3.1 we were using
> > $url.resource() as opposed to $url.themeResource()...  because now the
> > resources are copied across when you customize a theme so instead of
> > using the original theme resource, you would use your own copy.
> >
> > Just checked 'sotto' & 'brushmetal' theme (both shipped with roller)
> > and they use $url.resource() and not $url.themeResource().
> Allen G:
> > yes, that is correct and it's *imperative* that you use $url.resource()
> > this way to prevent pointers to shared resources.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/roller/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[Roller-JIRA] Updated: (ROL-1406) Optional themes use incorrect $url method

Posted by "David Johnson (JIRA)" <no...@atlassian.com>.
     [ http://opensource.atlassian.com/projects/roller/browse/ROL-1406?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Johnson updated ROL-1406:
-------------------------------

    Fix Version/s: 3.1.1

> Optional themes use incorrect $url method
> -----------------------------------------
>
>                 Key: ROL-1406
>                 URL: http://opensource.atlassian.com/projects/roller/browse/ROL-1406
>             Project: Roller
>          Issue Type: Bug
>          Components: Theme specific
>    Affects Versions: 3.1
>            Reporter: David Johnson
>            Assignee: Roller Unassigned
>             Fix For: 3.1.1
>
>
> From Sedat:
> > In roller 3.1 oprional themes, $url.resource(String
> > file)  macro is used to get the resources
> > (images,styles etc) of a theme. But this will not
> > work, and the appearance of theme is not seen
> > correctly. I change the usage of $url.resource(String
> > file) with $url.themeResource(String theme, String
> > file) macro and the themes are working correctly now.
> Richard Jones wrote:
> > Correct me if I'm wrong but I thought that now in 3.1 we were using
> > $url.resource() as opposed to $url.themeResource()...  because now the
> > resources are copied across when you customize a theme so instead of
> > using the original theme resource, you would use your own copy.
> >
> > Just checked 'sotto' & 'brushmetal' theme (both shipped with roller)
> > and they use $url.resource() and not $url.themeResource().
> Allen G:
> > yes, that is correct and it's *imperative* that you use $url.resource()
> > this way to prevent pointers to shared resources.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/roller/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira