You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@archiva.apache.org by "Maria Odea Ching (JIRA)" <ji...@codehaus.org> on 2009/12/17 09:35:55 UTC

[jira] Created: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Query using Group ID field in audit log report uses '/' as a separator instead of '.'
-------------------------------------------------------------------------------------

                 Key: MRM-1304
                 URL: http://jira.codehaus.org/browse/MRM-1304
             Project: Archiva
          Issue Type: Bug
          Components: web application
            Reporter: Maria Odea Ching


For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Updated: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Maria Odea Ching (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Maria Odea Ching updated MRM-1304:
----------------------------------

    Component/s: reporting

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Maria Odea Ching
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Assigned: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Maria Odea Ching (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Maria Odea Ching reassigned MRM-1304:
-------------------------------------

    Assignee: Maria Odea Ching

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Maria Odea Ching
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Updated: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Adelita L. Padilla (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Adelita L. Padilla updated MRM-1304:
------------------------------------

    Attachment: MRM-1304.patch

Fix and test attached.

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Maria Odea Ching
>             Fix For: 1.3.1
>
>         Attachments: MRM-1304.patch
>
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Updated: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Maria Odea Ching (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Maria Odea Ching updated MRM-1304:
----------------------------------

    Fix Version/s: 1.3.1

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Maria Odea Ching
>             Fix For: 1.3.1
>
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Commented: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Brett Porter (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=209174#action_209174 ] 

Brett Porter commented on MRM-1304:
-----------------------------------

the patch is missing the method name for the test case (Which is also disabled - have you checked it works on firefox?)

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Maria Odea Ching
>             Fix For: 1.3.1
>
>         Attachments: MRM-1304.patch, MRM-1304.patch, MRM-1304.patch, Updated_MRM-1304.patch
>
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Updated: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Adelita L. Padilla (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Adelita L. Padilla updated MRM-1304:
------------------------------------

    Attachment: MRM-1304.patch

Attached is the revised fix.

Thanks!

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Maria Odea Ching
>             Fix For: 1.3.1
>
>         Attachments: MRM-1304.patch, MRM-1304.patch
>
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Commented: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Brett Porter (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=209101#action_209101 ] 

Brett Porter commented on MRM-1304:
-----------------------------------

the patch looks fine, and thanks for adding a test!

I would only nitpick a couple of things:
- please fix the formatting in the patch (http://archiva.apache.org/developers/how-we-do-things.html). You might like to get the IDE code settings from the Maven site and use that to reformat your changes
- modifiedGroup can be created in the block - there's no reason for it to exist outside the if (nor be initialised separately).

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Maria Odea Ching
>             Fix For: 1.3.1
>
>         Attachments: MRM-1304.patch, MRM-1304.patch, Updated_MRM-1304.patch
>
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Updated: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Adelita L. Padilla (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Adelita L. Padilla updated MRM-1304:
------------------------------------

    Attachment: MRM-1304.patch

Attached revised patch for the fix and test. 

Fixed formatting and removed the extra line for modifiedGroupId.

Thanks! 

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Maria Odea Ching
>             Fix For: 1.3.1
>
>         Attachments: MRM-1304.patch, MRM-1304.patch, MRM-1304.patch, Updated_MRM-1304.patch
>
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Closed: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Brett Porter (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brett Porter closed MRM-1304.
-----------------------------

    Resolution: Fixed
      Assignee: Brett Porter  (was: Maria Odea Ching)

I've applied this, the fix looks fine.

I had to modify the Selenium test a bit to work, however.

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Brett Porter
>             Fix For: 1.3.1
>
>         Attachments: MRM-1304.patch, MRM-1304.patch, MRM-1304.patch, Updated_MRM-1304.patch
>
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Updated: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Adelita L. Padilla (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Adelita L. Padilla updated MRM-1304:
------------------------------------

    Attachment: Updated_MRM-1304.patch

Attached updated patch. (Updated_MRM-1304.patch)
* Fix and Test included


Please disregard the 2 previous attachments.

Thanks!

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Maria Odea Ching
>             Fix For: 1.3.1
>
>         Attachments: MRM-1304.patch, MRM-1304.patch, Updated_MRM-1304.patch
>
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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

        

[jira] Commented: (MRM-1304) Query using Group ID field in audit log report uses '/' as a separator instead of '.'

Posted by "Brett Porter (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MRM-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=209100#action_209100 ] 

Brett Porter commented on MRM-1304:
-----------------------------------

It's best to use the same name, since JIRA greys out the older ones :)

> Query using Group ID field in audit log report uses '/' as a separator instead of '.'
> -------------------------------------------------------------------------------------
>
>                 Key: MRM-1304
>                 URL: http://jira.codehaus.org/browse/MRM-1304
>             Project: Archiva
>          Issue Type: Bug
>          Components: reporting, web application
>            Reporter: Maria Odea Ching
>            Assignee: Maria Odea Ching
>             Fix For: 1.3.1
>
>         Attachments: MRM-1304.patch, MRM-1304.patch, Updated_MRM-1304.patch
>
>
> For groupIds with '.' (e.g. com.mycompany), you should use '/' instead (e.g. com/mycompany) as the artifact details are stored as artifact paths. 

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