Goto /groups/mine and apply a filter that returns no groups.

The message "You haven't joined any groups yet" could be confusing.

Comments

lightsurge’s picture

Status: Active » Needs review

I changed the empty text of this view to php format with:

<?php
global $user;
$view = views_get_current_view();
  if (count($view->exposed_input) && !empty($user->og_groups)) {
    return "Sorry, no groups were found.  Please adjust your search criteria and try again.";
}
else {
return "You haven't joined any groups yet.";
}
?>
lightsurge’s picture

Title: If you filter the My Groups page and there are no result you are told: "You haven't joined any groups yet" » When filtering returns 0 entries, empty text for group directory and group content views is inaccurate and misleading.
Status: Needs review » Active

Okay it looks like the empty text for all the views with exposed filters displays an inaccurate message along the lines of 'There are no [blogs,discussions,documents etc] in this group yet.' Not just the group directory.

Adjusting the title of this as I guess there's more work to do.

lightsurge’s picture

Status: Active » Needs work

I would be tempted to change all the group content views empty text to something like "No content found" in the case of a search, and "There is no blog content in this group" in the case of it being empty without a search.

Except for the case of the group directory (see #1).

Otherwise in the empty text we'd have to run an unfiltered query to check whether or not there would've been content if the user hadn't used a filter, in order to compare the two cases of 'empty'...

This seems too much messing about.

So I would just do this for each group content-type view:

<?php
$view = views_get_current_view();
  if (count($view->exposed_input)) {
    return "Sorry, no [blog entries] were found.  Please adjust your search criteria and try again.";
}
else {
return "There are no [blog entries in this group.";
}
?>
ezra-g’s picture

I think this is active, since we need to suggest better empty text and then provide a patch to implement it.

mstef’s picture

Status: Needs work » Active

I agree - I think they should all be changed.

Suggestions?

lsolesen’s picture

Version: » 6.x-2.x-dev
Issue summary: View changes
Status: Active » Closed (won't fix)
lsolesen’s picture

Status: Closed (won't fix) » Active

Closed by mistake. This might still be a valid issue. Please close it with the solution, if you already solved it.