Project

General

Profile

Bug #8876

Groups cannot select cuny.is URL that matches their group slug exactly

Added by scott voth almost 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
cuny.is
Target version:
Start date:
2017-10-31
Due date:
% Done:

0%

Estimated time:

Description

On our prod site, I tried to change a groups cuny.is shortlink. Original was group-omeka. I tried to change to omeka. Got message that it was already used - but when I go the cuny.is/omeka - it goes to CAC home page.

omeka group.png (72.3 KB) omeka group.png scott voth, 2017-10-31 08:37 PM

History

#1 Updated by Boone Gorges almost 2 years ago

  • Subject changed from CUNY.IS for groups to Groups cannot select cuny.is URL that matches their group slug exactly
  • Status changed from New to Assigned
  • Assignee changed from Marilyn Weber to Boone Gorges
  • Target version set to 1.12.1

Thanks, Scott. It appears that there's a safeguard in the plugin that prevents you from selecting a URL that exactly matches a group's slug (ie, if there's a group commons.gc.cuny.edu/groups/foo, then cuny.is/foo is protected). This made sense when all cuny.is URLs pointed to users - it would be confusing for a user to essentially hijack a group's URL - but it doesn't make a lot of sense anymore. I'll fix this.

#2 Updated by Boone Gorges almost 2 years ago

  • Status changed from Assigned to Reporter Feedback

This required two fixes:

https://github.com/cuny-academic-commons/cac/commit/ba37c859c34462ed61648c5a58a32809aaecc430 fixes the problem described above - namely, that groups should be able to claim the cuny.is URL corresponding to their Commons slug.

https://github.com/cuny-academic-commons/cac/commit/6d58d6c41b8499b66fd27a991b98e7b2683c9029 adds the ability for groups to claim the cuny.is URL of a Commons blog, as long as that blog is the groupblog of the group in question. So the Omeka group is allowed to have 'omeka' even though omeka.commons exists, since omeka.commons is linked to the Omeka group.

Because this is a big feature for 1.12, I've gone ahead and pushed these changes out as hotfixes. Scott, can you please verify?

#3 Updated by scott voth almost 2 years ago

Yes it is working now. Thanks.

#4 Updated by Boone Gorges almost 2 years ago

  • Status changed from Reporter Feedback to Resolved

Thanks for confirming!

Also available in: Atom PDF