Problem/Motivation

User module ships two views that were originally shipped with views in contrib - 'Who's online' and 'Who's new'.

Who's online has some privacy considerations even for a community site, and who's new is liable to show spam accounts unless user accounts are pre-moderated.

I'm not aware of any sites that use either view, and there are plenty of examples of views both in core and contrib these days, so that core modules don't need to ship 'demo' views.

These views aren't just dormant. They show up in the views UI unless you explicitly delete them. Also on every cold cache request they have to be loaded from the database and written to cache. Writing all the views that Umami ships with to the cache bin takes 80ms and there about 20 views shipped with umami. So it's potentially 1-4ms on every site that doesn't delete the views. Very small but non-zero.

See this from the Gander dashboard:

Screenshot of a trace showing 20 views config objects being written in a setMultiple call to the config cache backend

https://gander.tag1.io/explore?schemaVersion=1&panes=%7B%22ac5%22:%7B%22...

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
Screenshot from 2026-05-29 09-50-10.png203.81 KBcatch

Issue fork drupal-3592672

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

catch created an issue. See original summary.

catch’s picture

Status: Active » Needs work

Made a start on removals and fixed a couple of obvious tests, expecting more test failures.

catch’s picture

Not that bad - a couple of views UI tests that are using the who's online view, can likely switch that to a different view that's not going anywhere (did that for one test already).

InstallerExistingConfigMultilingualTest needs a bit more work - can probably also switch the view that's used, but also there's a fixture of the who's online view used in that test that would be swapped out too.

danielveza made their first commit to this issue’s fork.

danielveza’s picture

Status: Needs work » Needs review

Fixed the remaining tests, and removed the stale versions of these views in the translation tests. Ready for review. I believe the fail is unrelated.

catch’s picture

Status: Needs review » Reviewed & tested by the community

Looks great. Net approx 1000 lines removed.

catch’s picture

Status: Reviewed & tested by the community » Needs work

Needs a rebase.

gábor hojtsy’s picture

danielveza’s picture

Status: Needs work » Reviewed & tested by the community

Rebased, back to green

godotislate’s picture

Couple of Qs on the MR.

godotislate’s picture

Status: Reviewed & tested by the community » Needs review
danielveza’s picture

Adressed comments. IMO this is ready to be RTBC again, but I'll let someone else do that

godotislate’s picture

Status: Needs review » Reviewed & tested by the community

Yes, changes needed were minimal, so fine to go back to RTBC.

  • longwave committed 906ccf1d on main
    task: #3592672 Remove the who's new and who's online views
    
    By: catch
    By...
longwave’s picture

Status: Reviewed & tested by the community » Fixed

Yep these are obsolete and probably should have been removed a long time ago.

Committed and pushed 906ccf1db91 to main. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.