canvaslms 5.2__py3-none-any.whl → 5.3__py3-none-any.whl

canvaslms/cli/modules.nw

@@ -501,16 +501,27 @@ def get_item_modules(course, item_type, item_id):
 
 The matching logic differs by item type.
 For assignments, Canvas stores the assignment ID in [[content_id]].
-For pages, Canvas stores the URL slug in [[page_url]].
+For pages, Canvas stores the URL slug in [[page_url]]. However, Canvas may
+redirect old URLs to new ones, so we need to resolve the URL before comparing
+(see [[<<check if page item matches by resolving url>>]]).
 <<check if item matches and add module to list>>=
 if item_type == 'Assignment':
   if hasattr(item, 'content_id') and item.content_id == item_id:
     modules.append(module.name)
     break
 elif item_type == 'Page':
-  if hasattr(item, 'page_url') and item.page_url == item_id:
-    modules.append(module.name)
-    break
+  if hasattr(item, 'page_url'):
+    if item.page_url == item_id:
+      modules.append(module.name)
+      break
+    # Check if URLs resolve to same page (handles redirects)
+    try:
+      resolved_page = course.get_page(item.page_url)
+      if resolved_page.url == item_id:
+        modules.append(module.name)
+        break
+    except Exception:
+      pass
 @
 
 \subsection{Updating module membership based on patterns}
@@ -575,6 +586,21 @@ for module in all_modules:
 @
 
 We search the module's items to find if our item is currently present.
+
+For pages, we need to handle URL redirects. Canvas may store an old URL in the
+module item that redirects to the canonical URL we're checking against. For
+example, a page might have been added to a module with URL
+\enquote{page-on-campus+zoom}, but Canvas later changed the canonical URL to
+\enquote{page-on-zoom+campus}. The module item still stores the old URL, but
+when we fetch the page using [[course.get_page()]], Canvas returns the page
+with its new canonical URL.
+
+Without handling this, the direct comparison [[item.page_url == item_id]] would
+fail, causing the page to be added to the module again (creating duplicates).
+
+To detect this, we fetch the page using the module item's [[page_url]] and
+compare its canonical [[url]] attribute with [[item_id]]. This adds an API call
+per page item, but only when the direct comparison fails.
 <<find current module item if present>>=
 current_item = None
 try:
@@ -586,13 +612,29 @@ try:
         current_item = item
         break
     elif item_type == 'Page':
-      if hasattr(item, 'page_url') and item.page_url == item_id:
-        current_item = item
-        break
+      if hasattr(item, 'page_url'):
+        <<check if page item matches by resolving url>>
 except Exception:
   pass
 @
 
+When comparing page URLs, we first try a direct match. If that fails, we fetch
+the page to get its canonical URL and compare that. This handles the case where
+Canvas has created a redirect from an old URL to a new one.
+<<check if page item matches by resolving url>>=
+if item.page_url == item_id:
+  current_item = item
+  break
+# URLs don't match directly; check if they resolve to same page
+try:
+  resolved_page = course.get_page(item.page_url)
+  if resolved_page.url == item_id:
+    current_item = item
+    break
+except Exception:
+  pass  # Page doesn't exist or can't be fetched
+@
+
 Finally, we add or remove based on comparing current vs desired state.
 When adding, we use [[create_module_item]] with the appropriate type and ID.
 When removing, we call [[delete()]] on the module item object.

canvaslms/cli/modules.py

@@ -209,9 +209,18 @@ def get_item_modules(course, item_type, item_id):
                         modules.append(module.name)
                         break
                 elif item_type == "Page":
-                    if hasattr(item, "page_url") and item.page_url == item_id:
-                        modules.append(module.name)
-                        break
+                    if hasattr(item, "page_url"):
+                        if item.page_url == item_id:
+                            modules.append(module.name)
+                            break
+                        # Check if URLs resolve to same page (handles redirects)
+                        try:
+                            resolved_page = course.get_page(item.page_url)
+                            if resolved_page.url == item_id:
+                                modules.append(module.name)
+                                break
+                        except Exception:
+                            pass
         except Exception:
             # Skip modules we can't access
             pass
@@ -254,9 +263,18 @@ def update_item_modules(course, item_type, item_id, module_regexes):
                         current_item = item
                         break
                 elif item_type == "Page":
-                    if hasattr(item, "page_url") and item.page_url == item_id:
-                        current_item = item
-                        break
+                    if hasattr(item, "page_url"):
+                        if item.page_url == item_id:
+                            current_item = item
+                            break
+                        # URLs don't match directly; check if they resolve to same page
+                        try:
+                            resolved_page = course.get_page(item.page_url)
+                            if resolved_page.url == item_id:
+                                current_item = item
+                                break
+                        except Exception:
+                            pass  # Page doesn't exist or can't be fetched
         except Exception:
             pass
         should_be_in = module.id in matching_module_ids

canvaslms/cli/pages.nw

@@ -523,6 +523,11 @@ title to find the page. This is the most common case when creating or updating
 pages from Markdown files: the user specifies a title in the front matter and
 expects the command to find or create a page with that title.
 
+This behavior is critical for safety. Without it, the command would fall back to
+filter-based matching with the default [[-p '.*']] pattern, which matches
+\emph{all} pages in the course. The result would be catastrophic: every page
+would be overwritten with the content from the file.
+
 We use exact title matching (not regex) to avoid accidentally updating multiple
 pages. If exactly one page matches, we update it. If no pages match and
 [[--create]] is specified, we create a new page. If multiple pages match, we

{canvaslms-5.2.dist-info → canvaslms-5.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: canvaslms
-Version: 5.2
+Version: 5.3
 Summary: Command-line interface to Canvas LMS
 License-Expression: MIT
 License-File: LICENSE

{canvaslms-5.2.dist-info → canvaslms-5.3.dist-info}/RECORD

@@ -20,9 +20,9 @@ canvaslms/cli/grade.nw,sha256=ms7sBiGRPbK0CJLKxxYx_CDY9LMWgKxUuf2M3FYepMs,2197
 canvaslms/cli/grade.py,sha256=b7YkFIz64oXzcV2FcptpYJphevuCU3cdx9CilZHcG_A,662
 canvaslms/cli/login.nw,sha256=93LyHO_LXL1WdEvMg3OLhWulgkdoO8pfjYZVLwUbX4I,4419
 canvaslms/cli/login.py,sha256=GSO_wIT4TvCOwxaNW0VfUqjrkzsHvnImuSH4SdCu92M,2751
-canvaslms/cli/modules.nw,sha256=Wuh1aFwZJDEUAEzzgALnc-YVXNdRgRI819zdV7oIB_E,21864
-canvaslms/cli/modules.py,sha256=n1MfeZfjPTKyFwgluHwBqW8PKHwCGktWzYA6nuFLeL0,11483
-canvaslms/cli/pages.nw,sha256=MQOdVmMpERc5xUmnnTeEACZwVA6w8PcYACZvpmRKmZE,28648
+canvaslms/cli/modules.nw,sha256=VZkb9u9VGmXoCyJpPyByFrOanqbKSLnIRPxODSfldxo,23689
+canvaslms/cli/modules.py,sha256=7pnKU6ex6zH4OC2SxLMn4uzc0cDl3pAVFcR9WJi317c,12426
+canvaslms/cli/pages.nw,sha256=njm6oQT22ryI8Z7O62d-qivQjbMX_je5OwhgHEvtVUg,28934
 canvaslms/cli/pages.py,sha256=lW8DpMeyQQoVcu7ztSj_PdW-oUJC0HSh5mDeo8BuaRc,27713
 canvaslms/cli/quizzes.nw,sha256=nO0lAJLpE03fwPTV8wCC0Zr65p7J5x2H7J0OU8KW5CA,203138
 canvaslms/cli/quizzes.py,sha256=gx-SPm0rI2XNVUl5Qm7s4UHvsvitwFyrNh35J3Lay9Y,152014
@@ -57,8 +57,8 @@ canvaslms/hacks/attachment_cache.py,sha256=LcOZqaa6jPrEJWUD-JYN5GTc3bxCbv2fr_vqu
 canvaslms/hacks/canvasapi.nw,sha256=ixmIHn4tgy-ZKtQ1rqWSw97hfY2m0qtGX0de2x89lwA,136470
 canvaslms/hacks/canvasapi.py,sha256=A-r48x7gO6143_QkuZ8n6EW66i-a2AXXr7X7oeehOAU,31868
 canvaslms/hacks/test_hacks.py,sha256=JSJNvZqHu1E_s51HsPD7yr1gC-R-xVe-tuMMAKU9Gj8,66709
-canvaslms-5.2.dist-info/METADATA,sha256=QxMvCwVUPBhVxceECZCQMa69iYbtY_BoBCf1J2ZAdoM,5978
-canvaslms-5.2.dist-info/WHEEL,sha256=zp0Cn7JsFoX2ATtOhtaFYIiE2rmFAD4OcMhtUki8W3U,88
-canvaslms-5.2.dist-info/entry_points.txt,sha256=lyblfkLbodN5yb7q1c6-rwIoJPV-ygXrB9PYb5boHXM,48
-canvaslms-5.2.dist-info/licenses/LICENSE,sha256=N_TKsbzzD5Ax5fWJqEQk9bkwtf394MJkNeFld4HV6-E,1074
-canvaslms-5.2.dist-info/RECORD,,
+canvaslms-5.3.dist-info/METADATA,sha256=rvvX57QhCQT6AtrYaPIaBZ5JJ1KDay6O-zOSd61f2SA,5978
+canvaslms-5.3.dist-info/WHEEL,sha256=zp0Cn7JsFoX2ATtOhtaFYIiE2rmFAD4OcMhtUki8W3U,88
+canvaslms-5.3.dist-info/entry_points.txt,sha256=lyblfkLbodN5yb7q1c6-rwIoJPV-ygXrB9PYb5boHXM,48
+canvaslms-5.3.dist-info/licenses/LICENSE,sha256=N_TKsbzzD5Ax5fWJqEQk9bkwtf394MJkNeFld4HV6-E,1074
+canvaslms-5.3.dist-info/RECORD,,