/lastseen is frequently broken for older players

Issue #2 resolved
Riot created an issue

this functionality is much better handled by an exec plugin; please remove the command from Echelon

Comments (5)

  1. Tyler Waugh

    It fails when people haven't logged on since Echelon was implemented. Instead of removing it we should load in the missing data. That would be much faster than reverse fgreping through the logs every time.

  2. Riot reporter

    Who said anything about reverse fgrepping? I'm not insane. Exec scripts are perfectly capable of querying the echelon database as well; however, they can also bring into play more sophisticated query methods, such as checking the filesystem flags on the player dat files, checking admin last login times to the server, and even querying external APIs over http or direct sockets for information; moreover it would add a lot of flexibility over having this locked away in a plugin that has to be recompiled just to make minor changes.

  3. Tyler Waugh

    Well that's a good reason then. The old information needs to be loaded to fix the lastseen errors for older players.

  4. Tyler Waugh

    Updated, haven't changed the jar symlink. I'm not around to make sure things are working so not going to change the symlink yet.

  5. Log in to comment