libertree-model 0.9.11 → 0.9.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 9a84aeab884f0cc3db24026c3b4063a2a7ce7ece
-  data.tar.gz: f29e7defa9f0dc6cc60885bc4ee213bd737f65f2
+SHA256:
+  metadata.gz: 1d084f6691620280affdb0defc6e7ab1d2f7c0b3a7966add5adee31907afb088
+  data.tar.gz: 7a521d8ba93a593d3d07b8fcf91489b45a6df057d7a557777d72ced929fdd9d8
 SHA512:
-  metadata.gz: 4bfa2f35c02ee6f76aa8701ed525593fcd18433f19fd9bfa2dcd21bde958f84b35f0716ff91e2c38b79d48d2331c4150bcd65ac49c7dafae275835382189c46c
-  data.tar.gz: ac2e3ab424a0d6daee34cfd9c0db9cab87c77c7d42b9be887211306f54379b7592ddbef9b3be77aa5a5ea679bca37079b97a4140fbcbc54b6c5bdf9347f71bdb
+  metadata.gz: 8dc52396d815bd7e0b76d3212bc30151ded903bb1a285b2858ed1482857191eaf2cea655604c39cdde58cb07c2731f9c8a425e412d67c262c726c438c142e247
+  data.tar.gz: 3c1d4b88e5962951d6e438af844527c5ba5ffb2f1f86abad523d33f139642d6c7f8da158fea829021e3919e38e10e5ac03fa01a3aed9f3e0e573c017559d85f3

data/lib/libertree/model/account.rb

@@ -42,8 +42,13 @@ module Libertree
 
       def self.authenticate_db(creds)
         if creds['password_reset_code'].to_s
-          account = Account.where(%{password_reset_code = ? AND NOW() <= password_reset_expiry},
-                                  creds['password_reset_code'].to_s).first
+          account = Account.where(
+            Sequel.lit(
+              %{password_reset_code = ? AND NOW() <= password_reset_expiry},
+              creds['password_reset_code'].to_s
+            )
+          ).first
+
           if account
             return account
           end
@@ -286,7 +291,9 @@ module Libertree
       end
 
       def invitations_not_accepted
-        Invitation.where("inviter_account_id = ? AND account_id IS NULL", self.id).order(:id).all
+        Invitation.where(
+          Sequel.lit("inviter_account_id = ? AND account_id IS NULL", self.id)
+        ).order(:id).all
       end
 
       def new_invitation
@@ -362,7 +369,10 @@ module Libertree
       # @return [Boolean] true iff password reset was successfully set up
       def self.set_up_password_reset_for(email)
         # TODO: Don't allow registration of accounts with the same email but different case
-        account = self.where('LOWER(email) = ?', email.downcase).first
+        account = self.where(
+          Sequel.lit('LOWER(email) = ?', email.downcase)
+        ).first
+
         if account
           account.password_reset_code = SecureRandom.hex(16)
           account.password_reset_expiry = Time.now + 60 * 60

data/lib/libertree/model/chat-message.rb

@@ -67,8 +67,13 @@ module Libertree
       def self.mark_seen_between(account, member_id)
         return  if account.nil?
 
-        self.where("from_member_id = ? AND to_member_id = ?", member_id.to_i, account.member.id).
-          update(seen: true)
+        self.where(
+          Sequel.lit(
+            "from_member_id = ? AND to_member_id = ?",
+            member_id.to_i,
+            account.member.id
+          )
+        ).update(seen: true)
       end
     end
   end

data/lib/libertree/model/has-searchable-text.rb

@@ -9,10 +9,12 @@ module Libertree
         else
           dict = 'english'
         end
-        self.where("(to_tsvector('simple', text) || to_tsvector('english', text)) @@ plainto_tsquery('#{dict}', ?)", q).
-          reverse_order(:time_created).
-          limit(limit.to_i).
-          all
+
+        self.where(
+          Sequel.lit("(to_tsvector('simple', text) || to_tsvector('english', text)) @@ plainto_tsquery('#{dict}', ?)", q)
+        ).reverse_order(:time_created)
+        .limit(limit.to_i)
+        .all
       end
     end
   end

data/lib/libertree/model/job.rb

@@ -45,7 +45,12 @@ module Libertree
 
       # @return [Job] nil if no job was reserved
       def self.reserve(tasks)
-        job = self.where("task IN ? AND pid IS NULL AND tries < #{MAX_TRIES} AND time_to_start <= NOW()", tasks).order(:time_to_start).limit(1).first
+        job = self.where(
+          Sequel.lit("task IN ? AND pid IS NULL AND tries < #{MAX_TRIES} AND time_to_start <= NOW()", tasks)
+        ).order(:time_to_start)
+        .limit(1)
+        .first
+
         return nil  if job.nil?
 
         self.where({ id: job.id, pid: nil }).
@@ -72,20 +77,22 @@ module Libertree
         params = args[1..-1]
 
         self.where(
-          query + %{
-            AND time_finished IS NULL
-            AND tries < ?
-          },
-          *params,
-          MAX_TRIES
+          Sequel.lit(
+            query + %{
+              AND time_finished IS NULL
+              AND tries < ?
+            },
+            *params,
+            MAX_TRIES
+          )
         )
       end
 
       def self.unfinished(task=nil)
         if task
-          self.where("task = ? AND time_finished IS NULL", task).all
+          self.where(Sequel.lit("task = ? AND time_finished IS NULL", task)).all
         else
-          self.where("time_finished IS NULL").all
+          self.where(Sequel.lit("time_finished IS NULL")).all
         end
       end
     end

data/lib/libertree/model/member.rb

@@ -80,18 +80,23 @@ module Libertree
         if local
           self.qualify.
             join(:accounts, :id=>:account_id).
-            where(:accounts__username => username).
+            where(Sequel.qualify(:accounts, :username) => username).
             limit(1).
             first
         else
           # TODO: servers.name_given is no longer used.  Remove it after
           # migrating user rivers/contact lists etc.
-          self.qualify.
-            join(:servers, :id=>:server_id).
-            where(:members__username => username).
-            where(Sequel.or(:servers__domain => host, :servers__name_given => host)).
-            limit(1).
-            first
+          self.qualify
+            .join(:servers, :id=>:server_id)
+            .where(Sequel.qualify(:members, :username) => username)
+            .where(
+              Sequel.or(
+                Sequel.qualify(:servers, :domain) => host,
+                Sequel.qualify(:servers, :name_given) => host
+              )
+            )
+            .limit(1)
+            .first
         end
       end
 
@@ -100,7 +105,7 @@ module Libertree
       def self.with_display_name(name)
         self.qualify.
           join(:profiles, :member_id=>:id).
-          where(:profiles__name_display => name).
+          where(Sequel.qualify(:profiles, :name_display) => name).
           limit(1).
           first
       end

data/lib/libertree/model/node_subscription.rb

@@ -7,6 +7,7 @@ module Libertree
                  :subscribed ]
 
       many_to_one :node
+      set_primary_key :id
 
       def self.for(jid_or_host)
         return self  unless jid_or_host
@@ -14,7 +15,7 @@ module Libertree
         if jid_or_host.include?('@')
           self.where(jid: jid_or_host)
         else
-          host_pattern = self.where.escape_like(jid_or_host.to_s)
+          host_pattern = self.where(Sequel.lit('true')).escape_like(jid_or_host.to_s)
           self.where(Sequel.like(:jid, "%@#{host_pattern}"))
         end
       end

data/lib/libertree/model/notification.rb

@@ -46,8 +46,14 @@ module Libertree
       end
 
       def self.mark_seen_for_account_and_message(account, message)
-        self.where("account_id = ? AND data = ?", account.id, %|{"type":"message","message_id":#{message.id}}|).
-          update(seen: true)
+        self.where(
+          Sequel.lit(
+            "account_id = ? AND data = ?",
+            account.id,
+            %|{"type":"message","message_id":#{message.id}}|
+          )
+        ).update(seen: true)
+
         account.dirty
       end
 

data/lib/libertree/model/pool.rb

@@ -76,21 +76,29 @@ module Libertree
       def posts( opts = {} )
         limit = opts.fetch(:limit, 30)
         time = Time.at( opts.fetch(:time, Time.now.to_f) ).strftime("%Y-%m-%d %H:%M:%S.%6N%z")
-        time_clause = if opts[:newer]
-                        proc { time_created > time }
-                      else
-                        proc { time_created < time }
-                      end
-
-        res = Post.qualify.
-          join(:pools_posts, :post_id=>:id).
-          where(&time_clause).
-          where(:pool_id => self.id).
-          reverse_order(:posts__id).
-          limit(limit)
+        if opts[:newer]
+          time_clause = proc { time_created > time }
+        else
+          time_clause = proc { time_created < time }
+        end
+
+        order_by_updated = opts.fetch(:order_by_updated, false)
+        if order_by_updated
+          order = Sequel.lit("GREATEST(posts.time_commented, posts.time_updated)")
+        else
+          order = Sequel.qualify(:posts, :id)
+        end
+
+        res = Post.qualify
+        .join(:pools_posts, post_id: :id)
+        .where(&time_clause)
+        .where(pool_id: self.id)
+        .reverse_order(order)
+        .limit(limit)
 
         # optionally restrict to Internet visible posts
         res = res.where(visibility: 'internet')  if opts[:public]
+
         res
       end
 

data/lib/libertree/model/post.rb

@@ -96,7 +96,13 @@ module Libertree
       def mark_as_unread_by_all( options = {} )
         except_accounts = options.fetch(:except, [])
         if except_accounts.any?
-          DB.dbh[:posts_read].where('post_id = ? AND NOT account_id IN ?', self.id, except_accounts.map(&:id)).delete
+          DB.dbh[:posts_read].where(
+            Sequel.lit(
+              'post_id = ? AND NOT account_id IN ?',
+              self.id,
+              except_accounts.map(&:id)
+            )
+          ).delete
         else
           DB.dbh[ "DELETE FROM posts_read WHERE post_id = ?", self.id ].get
         end
@@ -351,47 +357,47 @@ module Libertree
         posts.
           qualify.
           left_outer_join(:posts_hidden,
-                          :posts_hidden__post_id => :posts__id,
-                          :posts_hidden__account_id => account.id).
-          where(:posts_hidden__post_id => nil)
+                          Sequel.qualify(:posts_hidden, :post_id) => Sequel.qualify(:posts, :id),
+                          Sequel.qualify(:posts_hidden, :account_id) => account.id).
+          where(Sequel.qualify(:posts_hidden, :post_id) => nil)
       end
 
       def self.read_by(account, posts=self)
         posts.
           qualify.
           join(:posts_read,
-               :posts_read__post_id => :posts__id,
-               :posts_read__account_id => account.id)
+               Sequel.qualify(:posts_read, :post_id) => Sequel.qualify(:posts, :id),
+               Sequel.qualify(:posts_read, :account_id) => account.id)
       end
 
       def self.unread_by(account, posts=self)
         posts.
           qualify.
           left_outer_join(:posts_read,
-                          :posts_read__post_id => :posts__id,
-                          :posts_read__account_id => account.id).
-          where(:posts_read__post_id => nil)
+                          Sequel.qualify(:posts_read, :post_id) => Sequel.qualify(:posts, :id),
+                          Sequel.qualify(:posts_read, :account_id) => account.id).
+          where(Sequel.qualify(:posts_read, :post_id) => nil)
       end
 
       def self.liked_by(member, posts=self)
         posts.
           qualify.
           join(:post_likes,
-               :post_likes__post_id => :posts__id,
-               :post_likes__member_id => member.id)
+               Sequel.qualify(:post_likes, :post_id) => Sequel.qualify(:posts, :id),
+               Sequel.qualify(:post_likes, :member_id) => member.id)
       end
 
       def self.without_liked_by(member, posts=self)
         posts.
           qualify.
           join(:post_likes,
-               :post_likes__post_id => :posts__id,
-               :post_likes__member_id => member.id)
+               Sequel.qualify(:post_likes, :post_id) => Sequel.qualify(:posts, :id),
+               Sequel.qualify(:post_likes, :member_id) => member.id)
       end
 
       def self.commented_on_by(member, posts=self)
         posts.
-          where(:posts__id => Comment.
+          where(Sequel.qualify(:posts, :id) => Comment.
                 select(:post_id).
                 distinct(:post_id).
                 where(:member_id => member.id))
@@ -399,7 +405,7 @@ module Libertree
 
       def self.without_commented_on_by(member, posts=self)
         posts.
-          exclude(:posts__id => Comment.
+          exclude(Sequel.qualify(:posts, :id) => Comment.
                   select(:post_id).
                   distinct(:post_id).
                   where(:member_id => member.id))
@@ -409,17 +415,17 @@ module Libertree
         posts.
           qualify.
           join(:post_subscriptions,
-               :post_subscriptions__post_id => :posts__id,
-               :post_subscriptions__account_id => account.id)
+               Sequel.qualify(:post_subscriptions, :post_id) => Sequel.qualify(:posts, :id),
+               Sequel.qualify(:post_subscriptions, :account_id) => account.id)
       end
 
       def self.without_subscribed_to_by(account, posts=self)
         posts.
           qualify.
           left_outer_join(:post_subscriptions,
-                          :post_subscriptions__post_id => :posts__id,
-                          :post_subscriptions__account_id => account.id).
-          where(:post_subscriptions__post_id => nil)
+                          Sequel.qualify(:post_subscriptions, :post_id) => Sequel.qualify(:posts, :id),
+                          Sequel.qualify(:post_subscriptions, :account_id) => account.id).
+          where(Sequel.qualify(:post_subscriptions, :post_id) => nil)
       end
 
       def self.filter_by_query(parsed_query, account, posts=self)
@@ -518,13 +524,19 @@ module Libertree
 
           # filter by simple terms first to avoid having to check so many posts
           # TODO: prevent empty arguments to to_tsquery
-          posts = posts.where(%{to_tsvector('simple', text)
-                               @@ (to_tsquery('simple', ?)
-                               && to_tsquery('simple', ?)
-                               && to_tsquery('simple', ?))},
-                             words[:negations].map{|w| "!#{w}" }.join(' & '),
-                             words[:requirements].join(' & '),
-                             words[:regular].join(' | '))
+          posts = posts.where(
+            Sequel.lit(
+              %{
+                to_tsvector('simple', text)
+                @@ (to_tsquery('simple', ?)
+                && to_tsquery('simple', ?)
+                && to_tsquery('simple', ?))
+              },
+              words[:negations].map{|w| "!#{w}" }.join(' & '),
+              words[:requirements].join(' & '),
+              words[:regular].join(' | ')
+            )
+          )
         end
 
         { 'visibility' => :visibility,
@@ -618,14 +630,16 @@ module Libertree
 
         get_comments = lambda do
           comments = Comment.on_post(post, viewing_account: viewing_account)
-          comment_likes = CommentLike.where('comment_id IN ?', comments.map(&:id)).reduce({}) do |hash, like|
+          comment_likes = CommentLike.where(
+            Sequel.lit('comment_id IN ?', comments.map(&:id))
+          ).reduce({}) { |hash, like|
             if hash[like.comment_id]
               hash[like.comment_id] << like
             else
               hash[like.comment_id] = [like]
             end
             hash
-          end
+          }
 
           comments.map do |comment|
             likes = if comment_likes[comment.id]
@@ -699,9 +713,14 @@ module Libertree
       end
 
       def pools_by_member(member_id)
-        Libertree::Model::Pool.qualify.
-          join(:pools_posts, :pools_posts__pool_id => :pools__id).
-          where(pools_posts__post_id: self.id, pools__member_id: member_id)
+        Libertree::Model::Pool.qualify
+          .join(
+            :pools_posts,
+            Sequel.qualify(:pools_posts, :pool_id) => Sequel.qualify(:pools, :id)
+          ).where(
+            Sequel.qualify(:pools_posts, :post_id) => self.id,
+            Sequel.qualify(:pools, :member_id) => member_id
+          )
       end
 
       def update_collection_status_for_member(member_id, pool_ids)

data/lib/libertree/model/profile.rb

@@ -18,7 +18,11 @@ module Libertree
       end
 
       def self.search(query)
-        self.where("(to_tsvector('simple', description) || to_tsvector('english', description)) @@ plainto_tsquery(?)", query).or("name_display ILIKE '%' || ? || '%'", query)
+        self.where(
+          Sequel.lit("(to_tsvector('simple', description) || to_tsvector('english', description)) @@ plainto_tsquery(?)", query)
+        ).or(
+          Sequel.lit("name_display ILIKE '%' || ? || '%'", query)
+        )
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: libertree-model
 version: !ruby/object:Gem::Version
-  version: 0.9.11
+  version: 0.9.16
 platform: ruby
 authors:
 - Pistos
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-18 00:00:00.000000000 Z
+date: 2020-07-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-oembed
@@ -17,112 +17,112 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.8
+        version: 0.13.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.8
+        version: 0.13.1
 - !ruby/object:Gem::Dependency
   name: gpgme
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.0.20
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.0.20
 - !ruby/object:Gem::Dependency
   name: pg
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.2.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.2.3
 - !ruby/object:Gem::Dependency
   name: sequel
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '5'
+        version: 5.34.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '5'
+        version: 5.34.0
 - !ruby/object:Gem::Dependency
   name: bcrypt-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.1
+        version: 3.1.5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.1
+        version: 3.1.5
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: 1.10.10
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: 1.10.10
 - !ruby/object:Gem::Dependency
   name: net-ldap
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.1
+        version: 0.16.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.1
+        version: 0.16.1
 - !ruby/object:Gem::Dependency
   name: parkdown-libertree
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.4.27
+        version: 1.4.26
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.4.27
+        version: 1.4.26
 description: Database library for Libertree
 email: 
 executables: []
@@ -190,8 +190,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.11
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Database library for Libertree