effective_learndash 0.9.0 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5137c9bb0e056c81ec1cf547ecf71fbbf07199b5a37d015fdadf6a7987857397
-  data.tar.gz: 76de32ccdad600bc2ea5b124d6686d8284451934c0257af68bb264f01926e0ea
+  metadata.gz: 59aa5f415bca0ab3c2cdfa34896bb1e25632bc517b752a3531324c397f7ae2df
+  data.tar.gz: a9749e2b97d0b6fda661fc5979d6127504e4cc921c42187766dabfe769d200d5
 SHA512:
-  metadata.gz: 8c873ed4d4ab4c21d61863f0ecf455d608df6fbc9afaada4044cee9276cd4ab027cfa78ca1b30db3748fe806ea89a6c5c513af7f6bdf14fefbfaa93a2e75ede4
-  data.tar.gz: 2d1e4a58440f11211fc27a67fdd4e923eedda689f1bbe99308c9a48b4eaa467128e35bbfb8f8c07e6c4ae376ff6ca24f7d23a755113df007e1a9db8d9dd707f2
+  metadata.gz: 1a7b1b381548714df217d64bc91022b70d4f535d9ff942e4cbeb78a57429781e86ad9348bcabce8e795686ee72bc31eba472335abe2102330ec19795306fd767
+  data.tar.gz: a5150ba0d68e755e6a91cda4a27fd4d90560f3446d77ef5ab0f82517cff0d6e6d459193d9e8c23fb9a0ef0ffb31402bdef7f240a708dfc3dea57367837ffd5a2

data/README.md

@@ -101,6 +101,25 @@ https://make.wordpress.org/core/2020/11/05/application-passwords-integration-gui
 
 and fill in the username/password details in config/initializers/effective_learndash.rb
 
+## LearnDash API compatibility
+
+The integration uses WordPress REST API v2 and LearnDash `ldlms/v2` endpoints.
+LearnDash 4 and 5 course-progress responses are supported: incoming underscore
+statuses such as `in_progress` are stored using the existing `in-progress` format.
+The `date_started_gmt` and `date_completed_gmt` fields are used when supplied;
+older responses without those fields retain their previous UTC interpretation.
+Both object and legacy array responses are supported for individual progress records.
+No database migration is required.
+
+Users, courses, user searches, and course-progress collections follow the
+`X-WP-TotalPages` response header with up to 100 records per request. An individual
+resource returning HTTP 404 is treated as missing; other unsuccessful GET requests
+raise `Effective::LearndashApi::Error`, rather than triggering user or enrollment
+creation after an authentication or server failure.
+
+See the [LearnDash 5 API changes](https://docs.nexcess.com/software/changelogs/learndash/sfwd-lms/5-0-0/)
+and [WordPress pagination reference](https://developer.wordpress.org/rest-api/using-the-rest-api/pagination/).
+
 ## Working with LearnDash
 
 Visit `/admin/learndash_courses` and Refresh the list of Courses.

data/app/datatables/effective_course_registrations_datatable.rb

@@ -20,7 +20,7 @@ class EffectiveCourseRegistrationsDatatable < Effective::Datatable
 
     actions_col(actions: []) do |registration|
       if registration.draft?
-        dropdown_link_to('Continue', effective_learndash.learndash_course_course_registration_build_path(registration.learndash_course, registration, registration.next_step), 'data-turbolinks' => false)
+        dropdown_link_to('Continue', effective_learndash.learndash_course_course_registration_build_path(registration.learndash_course, registration, registration.next_step), 'data-turbolinks' => false, 'data-turbo' => false)
         dropdown_link_to('Delete', effective_learndash.learndash_course_course_registration_path(registration.learndash_course, registration), 'data-confirm': "Really delete #{registration}?", 'data-method': :delete)
       else
         dropdown_link_to('Show', effective_learndash.learndash_course_course_registration_path(registration.learndash_course, registration))

data/app/models/effective/learndash_api.rb

@@ -2,6 +2,7 @@ require 'net/http'
 
 module Effective
   class LearndashApi
+    class Error < StandardError; end
 
     attr_accessor :url
     attr_accessor :username
@@ -22,7 +23,7 @@ module Effective
 
     # List users
     def users
-      get('/wp/v2/users')
+      get_collection('/wp/v2/users')
     end
 
     # Returns a WP Hash of User or nil
@@ -90,14 +91,14 @@ module Effective
 
     # List Courses
     def courses
-      get('/ldlms/v2/sfwd-courses')
+      get_collection('/ldlms/v2/sfwd-courses')
     end
 
     # List User Course Progress
     def user_enrollments(user)
       user = user_id(user) || raise('expected a user')
 
-      get("/ldlms/v2/users/#{user}/course-progress")
+      get_collection("/ldlms/v2/users/#{user}/course-progress")
     end
 
     # Helper methods for enrollments
@@ -195,33 +196,41 @@ module Effective
       raise('expected a symbol key') unless key.kind_of?(Symbol)
       raise('expected a value') unless value.present?
 
-      response = get(endpoint, { search: value, context: :edit })
-
-      collection = Array(
-        if response == false
-          nil
-        elsif response.kind_of?(Hash) && response.dig(:data, :status) == 404
-          nil
-        elsif response.kind_of?(Hash)
-          response
-        elsif response.kind_of?(Array)
-          response
-        else
-          raise("unexpected LearnDash API find_by response #{response}")
+      get_collection(endpoint, search: value, context: :edit).find { |data| data[key] == value }
+    end
+
+    # Follow WordPress pagination headers for collection endpoints only.
+    def get_collection(endpoint, params = nil)
+      collection = []
+      page = 1
+      total_pages = 1
+
+      loop do
+        response = get(endpoint, (params || {}).merge(per_page: 100, page: page)) do |http_response|
+          total_pages = http_response['X-WP-TotalPages'].to_i if http_response['X-WP-TotalPages'].present?
+        end
+
+        unless response.kind_of?(Array)
+          raise(Error, "Unexpected LearnDash API collection: #{endpoint}")
         end
-      )
 
-      resource = collection.find { |data| data[key] == value }
-      resource
+        collection.concat(response)
+        break if page >= total_pages
+
+        page += 1
+      end
+
+      collection
     end
 
     def get(endpoint, params = nil)
-      query = ('?' + params.compact.map { |k, v| "#{k}=#{v}" }.join('&')) if params.present?
+      query = ('?' + URI.encode_www_form(params.compact)) if params.present?
 
       uri = URI.parse(api_url + endpoint + query.to_s)
 
       http = Net::HTTP.new(uri.host, uri.port)
       http.use_ssl = (uri.scheme == 'https')
+      http.open_timeout = 10
       http.read_timeout = 10
 
       response = with_retries do
@@ -229,11 +238,10 @@ module Effective
         http.get(uri, headers)
       end
 
-      unless response.code.start_with?('2')
-        puts("Response code: #{response.code} #{response.body}") if Rails.env.development?
-        return false
-      end
+      return false if response.code == '404'
+      raise(Error, "Invalid LearnDash API response: HTTP #{response.code}") unless response.code.start_with?('2')
 
+      yield(response) if block_given?
       JSON.parse(response.body, symbolize_names: true)
     end
 
@@ -242,10 +250,11 @@ module Effective
 
       http = Net::HTTP.new(uri.host, uri.port)
       http.use_ssl = (uri.scheme == 'https')
+      http.open_timeout = 10
       http.read_timeout = 10
 
       response = with_retries do
-        puts("[POST] #{uri} #{params}") if Rails.env.development?
+        puts("[POST] #{uri}") if Rails.env.development?
         http.post(uri.path, (params || {}).to_json, headers)
       end
 

data/app/models/effective/learndash_enrollment.rb

@@ -106,16 +106,18 @@ module Effective
         steps_total: data[:steps_total],
       )
 
-      if (date = data[:date_started]).present?
+      date = data.key?(:date_started_gmt) ? data[:date_started_gmt] : data[:date_started]
+      if date.present?
         assign_attributes(date_started: Time.use_zone('UTC') { Time.zone.parse(date) })
       end
 
-      if (date = data[:date_completed]).present?
+      date = data.key?(:date_completed_gmt) ? data[:date_completed_gmt] : data[:date_completed]
+      if date.present?
         assign_attributes(date_completed: Time.use_zone('UTC') { Time.zone.parse(date) })
       end
 
       unless completed?
-        assign_attributes(progress_status: data[:progress_status])
+        assign_attributes(progress_status: data[:progress_status]&.tr('_', '-'))
       end
 
       true

data/app/views/effective/course_registrations/_dashboard.html.haml

@@ -12,7 +12,7 @@
 
   %p
     Please
-    = link_to("Continue registration for #{wizard.learndash_course}", effective_learndash.learndash_course_course_registration_build_path(wizard.learndash_course, wizard, wizard.next_step), 'data-turbolinks' => false, class: 'btn btn-primary')
+    = link_to("Continue registration for #{wizard.learndash_course}", effective_learndash.learndash_course_course_registration_build_path(wizard.learndash_course, wizard, wizard.next_step), 'data-turbolinks' => false, 'data-turbo' => false, class: 'btn btn-primary')
     or you can
     = link_to('Abandon course registration', effective_learndash.learndash_course_course_registration_path(wizard.learndash_course, wizard), 'data-confirm': "Really delete #{wizard}?", 'data-method': :delete, class: 'btn btn-danger')
     to register for another course.

data/lib/effective_learndash/version.rb

@@ -1,3 +1,3 @@
 module EffectiveLearndash
-  VERSION = '0.9.0'.freeze
+  VERSION = '0.11.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_learndash
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.11.0
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-08-26 00:00:00.000000000 Z
+date: 2026-10-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails