Any way to disable reactivity for Alethes Pages?

Spyridon · May 8, 2018, 9:58pm

Using one of the older pagination packages for one of my apps (Alethes Pages). Was wondering if there’s any way to easily disable the reactivity?

Reactivity is really unnecessary and is just causing additional load on server, especially for things like our logs page.

Hoping there’s a way to disable it, but I’m not sure considering it handles the subscriptions behind the scenes…

doctorpangloss · May 12, 2018, 3:40pm

Fork, and add reactive: false here:

github.com

alethes/meteor-pages/blob/d04eef808e388f4abf9f857e9ade1650913c6917/lib/pages.coffee#L603


delete @userSettings[cid]?.realFilters
delete @userSettings[cid]?.nPublishedPages


@setPerPage()


skip = (page - 1) * get "perPage"
skip = 0 if skip < 0


filters = get "filters"


options =
  sort: get "sort"
  fields: get "fields"
  skip: skip
  limit: get "perPage"


# Call the authentication function if it's supplied


if @auth?
  r = @auth.call @, skip, sub
  if !r

There will definitely be UX glitches, so test it out.

Spyridon · May 17, 2018, 10:54pm

The change did not seem to do anything at all?

doctorpangloss · May 18, 2018, 5:28pm

I’m sorry, it appears that the server doesn’t support that flag.

That code is written in some really inscrutable programmer-ese, so it’s hard for me to say exactly where the change needs to be made.

You actually have to replace calls like this…

github.com

alethes/meteor-pages/blob/d04eef808e388f4abf9f857e9ade1650913c6917/lib/pages.coffee#L654




self = @


# We need to call sub's added callback when a new document is added, however
# for the purposes of pagination we also need to include the index of each document.
#
# Furthermore, an added document might increase the index of other documents on this page.
#
# We therefore need to use the observe method to handle this.


handle = c.observe
  addedAt: _.bind ((sub, query, doc, at) ->
    return  if init
    #@log "#{doc.id} added at #{page}, #{at}"
    doc["_#{@id}_p"] = page
    doc["_#{@id}_i"] = at
    id = doc._id
    delete doc._id
    # Add to @Collection
    query().forEach (o, i) =>
      if i is at

… with find().fetch() and call sub.added on each resulting document. No more observe, no more sub.changed calls.

There are a few of them, so this is a bit involved, but not at all as error-proned as it may appear.