Вот так? Да, стало немного понятнее, но всё равно не гут.
$links[$value] = $this->isGuest()
? $value
: ($this->settings()->get('revote')
? ((!is_null($this->query()->exists()) && $value == $this->query()->exists())
? $value
: $link)
: $value);А ещё я заметил, что $this->query->exists() два раза вызывается, т.е. как я понял один и тот же запрос два раза выполняется без надобности. Вот к чему приводит экономия на спичках