Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
de572af675 | ||
|
|
234f663f04 | ||
|
|
8c3bacb2f2 | ||
|
|
20ea4f9291 | ||
|
|
bca6b874c6 | ||
|
|
f6caa6f016 | ||
|
|
6dae35001e | ||
|
|
4d52ef7f50 |
@@ -9,17 +9,19 @@ buttons) behind previous / jump / next navigation, with an optional cache button
|
|||||||
`requirements.txt`:
|
`requirements.txt`:
|
||||||
|
|
||||||
```
|
```
|
||||||
dpy_paginator @ git+ssh://git@git.rethinkstudios.io/rethink-public/dpy_paginator.git@v0.1.0
|
dpy_paginator @ git+ssh://git@git.rethinkstudios.io/rethink-public/dpy_paginator.git@v0.1.3
|
||||||
```
|
```
|
||||||
|
|
||||||
Direct:
|
Direct:
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
pip install "dpy_paginator @ git+ssh://git@git.rethinkstudios.io/rethink-public/dpy_paginator.git@v0.1.0"
|
pip install "dpy_paginator @ git+ssh://git@git.rethinkstudios.io/rethink-public/dpy_paginator.git@v0.1.3"
|
||||||
```
|
```
|
||||||
|
|
||||||
Requires `discord.py` (pulled transitively).
|
Requires `discord.py` (pulled transitively).
|
||||||
|
|
||||||
|
Drop the `@v0.1.3` suffix from the line above to install the latest unpinned.
|
||||||
|
|
||||||
## Basic usage
|
## Basic usage
|
||||||
|
|
||||||
Plain pages — just navigation:
|
Plain pages — just navigation:
|
||||||
@@ -116,7 +118,8 @@ await ButtonPaginator(pages, cache=cache, cache_sleep=1.0).start(ctx)
|
|||||||
```
|
```
|
||||||
|
|
||||||
Omit `cache` or pass `None`/`[]` and the button never appears. When set, `cache`
|
Omit `cache` or pass `None`/`[]` and the button never appears. When set, `cache`
|
||||||
must have one entry per rendered page.
|
must have one entry per rendered page — a shorter `cache` raises `ValueError` at
|
||||||
|
construction rather than failing with an `IndexError` mid-navigation.
|
||||||
|
|
||||||
## Constructor options
|
## Constructor options
|
||||||
|
|
||||||
@@ -138,4 +141,4 @@ in an embed). It may be sync or async.
|
|||||||
|
|
||||||
## Versioning
|
## Versioning
|
||||||
|
|
||||||
Tagged `vX.Y.Z`. Pin the tag in `requirements.txt`.
|
Releases are tagged `vX.Y.Z`. The install line above pins a release; drop the `@vX.Y.Z` suffix to install the latest unpinned. Pin deliberately for reproducible installs.
|
||||||
|
|||||||
+1
-1
@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
|
|||||||
|
|
||||||
[project]
|
[project]
|
||||||
name = "dpy_paginator"
|
name = "dpy_paginator"
|
||||||
version = "0.1.0"
|
version = "0.1.3"
|
||||||
description = "Button-navigated paginator for discord.py — config-free, injectable emojis, installable."
|
description = "Button-navigated paginator for discord.py — config-free, injectable emojis, installable."
|
||||||
requires-python = ">=3.10"
|
requires-python = ">=3.10"
|
||||||
dependencies = [
|
dependencies = [
|
||||||
|
|||||||
@@ -159,6 +159,8 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
cache_sleep: seconds to wait after priming before refreshing the view
|
cache_sleep: seconds to wait after priming before refreshing the view
|
||||||
"""
|
"""
|
||||||
super().__init__(timeout=timeout)
|
super().__init__(timeout=timeout)
|
||||||
|
if not pages:
|
||||||
|
raise ValueError("ButtonPaginator requires at least one page")
|
||||||
self.author_id: Optional[int] = author_id
|
self.author_id: Optional[int] = author_id
|
||||||
self.delete_message_after: bool = delete_message_after
|
self.delete_message_after: bool = delete_message_after
|
||||||
self.mentions_allowed = mentions_allowed or discord.AllowedMentions.all()
|
self.mentions_allowed = mentions_allowed or discord.AllowedMentions.all()
|
||||||
@@ -174,6 +176,11 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
|
|
||||||
total_pages, left_over = divmod(len(self.pages), self.per_page)
|
total_pages, left_over = divmod(len(self.pages), self.per_page)
|
||||||
self.max_pages: int = total_pages + (1 if left_over else 0)
|
self.max_pages: int = total_pages + (1 if left_over else 0)
|
||||||
|
if cache is not None and len(cache) < self.max_pages:
|
||||||
|
raise ValueError(
|
||||||
|
f"cache has {len(cache)} entries but there are {self.max_pages} pages; "
|
||||||
|
"cache needs one entry per page"
|
||||||
|
)
|
||||||
self._page_kwargs: Dict[str, Any] = self._fresh_kwargs()
|
self._page_kwargs: Dict[str, Any] = self._fresh_kwargs()
|
||||||
|
|
||||||
def _fresh_kwargs(self) -> Dict[str, Any]:
|
def _fresh_kwargs(self) -> Dict[str, Any]:
|
||||||
@@ -193,7 +200,7 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
|
|
||||||
async def interaction_check(self, interaction: Interaction) -> bool:
|
async def interaction_check(self, interaction: Interaction) -> bool:
|
||||||
"""restrict interaction to author_id when set"""
|
"""restrict interaction to author_id when set"""
|
||||||
if not self.author_id or self.author_id == interaction.user.id:
|
if self.author_id is None or self.author_id == interaction.user.id:
|
||||||
return True
|
return True
|
||||||
await interaction.response.send_message("You cannot interact with this menu.", ephemeral=True)
|
await interaction.response.send_message("You cannot interact with this menu.", ephemeral=True)
|
||||||
return False
|
return False
|
||||||
@@ -202,7 +209,7 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
"""return the content for a page index, wrapping out-of-range to 0"""
|
"""return the content for a page index, wrapping out-of-range to 0"""
|
||||||
if page_number < 0 or page_number >= self.max_pages:
|
if page_number < 0 or page_number >= self.max_pages:
|
||||||
self.current_page = 0
|
self.current_page = 0
|
||||||
return self.pages[self.current_page]
|
return self.get_page(0)
|
||||||
if self.per_page == 1:
|
if self.per_page == 1:
|
||||||
return self.pages[page_number]
|
return self.pages[page_number]
|
||||||
base = page_number * self.per_page
|
base = page_number * self.per_page
|
||||||
@@ -252,6 +259,14 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
if isinstance(value, discord.Attachment):
|
if isinstance(value, discord.Attachment):
|
||||||
value = await value.to_file()
|
value = await value.to_file()
|
||||||
self._page_kwargs["files"].append(value)
|
self._page_kwargs["files"].append(value)
|
||||||
|
elif key in ("embed", "embeds", "file", "files"):
|
||||||
|
# a wrong-typed embed/file would otherwise fall to the catch-all and
|
||||||
|
# be forwarded verbatim, colliding with the base embeds=[]/files=[]
|
||||||
|
# and raising an opaque TypeError from inside discord.py — reject it
|
||||||
|
# here with a clear, paginator-side message
|
||||||
|
raise ValueError(
|
||||||
|
f"page key {key!r} has unexpected type {type(value).__name__}"
|
||||||
|
)
|
||||||
else:
|
else:
|
||||||
self._page_kwargs[key] = value
|
self._page_kwargs[key] = value
|
||||||
elif isinstance(formatted_page, str):
|
elif isinstance(formatted_page, str):
|
||||||
@@ -273,27 +288,34 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
|
|
||||||
return self._page_kwargs
|
return self._page_kwargs
|
||||||
|
|
||||||
def update_buttons(self) -> None:
|
def update_buttons(self, nav: bool = True) -> None:
|
||||||
"""rebuild the action row for the current page state"""
|
"""rebuild the action row for the current page state
|
||||||
|
|
||||||
|
nav=False rebuilds with only the page's custom buttons and no navigation
|
||||||
|
items (prev/jump/cache/next) — a single-page result that still carries
|
||||||
|
custom buttons keeps them (and their live callbacks) without a nav row.
|
||||||
|
"""
|
||||||
self.clear_items()
|
self.clear_items()
|
||||||
|
|
||||||
self.previous_page.emoji = self.emojis["previous"]
|
if nav:
|
||||||
self.previous_page.disabled = self.current_page <= 0
|
self.previous_page.emoji = self.emojis["previous"]
|
||||||
self.add_item(self.previous_page)
|
self.previous_page.disabled = self.current_page <= 0
|
||||||
|
self.add_item(self.previous_page)
|
||||||
|
|
||||||
self.jump_button.label = self.page_text.format(self.current_page + 1, self.max_pages)
|
self.jump_button.label = self.page_text.format(self.current_page + 1, self.max_pages)
|
||||||
self.add_item(self.jump_button)
|
self.add_item(self.jump_button)
|
||||||
|
|
||||||
for button in self.current_page_buttons:
|
for button in self.current_page_buttons:
|
||||||
self.add_item(button)
|
self.add_item(button)
|
||||||
|
|
||||||
if self.cache:
|
if nav:
|
||||||
self.cache_button.emoji = self.emojis["cache"]
|
if self.cache:
|
||||||
self.add_item(self.cache_button)
|
self.cache_button.emoji = self.emojis["cache"]
|
||||||
|
self.add_item(self.cache_button)
|
||||||
|
|
||||||
self.next_page.emoji = self.emojis["next"]
|
self.next_page.emoji = self.emojis["next"]
|
||||||
self.next_page.disabled = self.current_page >= self.max_pages - 1
|
self.next_page.disabled = self.current_page >= self.max_pages - 1
|
||||||
self.add_item(self.next_page)
|
self.add_item(self.next_page)
|
||||||
|
|
||||||
async def _build_render_kwargs(self) -> Dict[str, Any]:
|
async def _build_render_kwargs(self) -> Dict[str, Any]:
|
||||||
"""build the edit-ready kwargs for the current page (buttons + attachments)"""
|
"""build the edit-ready kwargs for the current page (buttons + attachments)"""
|
||||||
@@ -363,21 +385,41 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
self.update_buttons()
|
self.update_buttons()
|
||||||
|
|
||||||
if self.max_pages < 2:
|
if self.max_pages < 2:
|
||||||
self.stop()
|
if self.current_page_buttons:
|
||||||
kwargs.pop("view", None)
|
# single page WITH custom buttons: keep the view live so the
|
||||||
|
# buttons' callbacks still fire; strip only the navigation row
|
||||||
|
self.update_buttons(nav=False)
|
||||||
|
else:
|
||||||
|
# single page, no custom buttons: no interactive row at all
|
||||||
|
self.stop()
|
||||||
|
kwargs.pop("view", None)
|
||||||
|
|
||||||
self.reset_files(kwargs)
|
self.reset_files(kwargs)
|
||||||
send_kwargs["ephemeral"] = self.ephemeral
|
|
||||||
|
|
||||||
if isinstance(obj, discord.Interaction):
|
if isinstance(obj, discord.Interaction):
|
||||||
|
# ephemeral is an interaction-response concept; only these paths accept it
|
||||||
if obj.response.is_done():
|
if obj.response.is_done():
|
||||||
self.message = await obj.followup.send(**kwargs, **send_kwargs)
|
self.message = await obj.followup.send(**kwargs, ephemeral=self.ephemeral, **send_kwargs)
|
||||||
else:
|
else:
|
||||||
await obj.response.send_message(**kwargs, **send_kwargs)
|
await obj.response.send_message(**kwargs, ephemeral=self.ephemeral, **send_kwargs)
|
||||||
self.message = await obj.original_response()
|
self.message = await obj.original_response()
|
||||||
elif isinstance(obj, Messageable):
|
elif isinstance(obj, Messageable):
|
||||||
|
# Messageable.send (a raw channel) has no ephemeral param — never pass it
|
||||||
self.message = await obj.send(**kwargs, **send_kwargs)
|
self.message = await obj.send(**kwargs, **send_kwargs)
|
||||||
else:
|
else:
|
||||||
raise TypeError(f"expected Interaction or Messageable, got {obj.__class__.__name__}")
|
raise TypeError(f"expected Interaction or Messageable, got {obj.__class__.__name__}")
|
||||||
|
|
||||||
return self.message
|
return self.message
|
||||||
|
|
||||||
|
async def on_timeout(self) -> None:
|
||||||
|
"""delete the message on timeout when delete_message_after is set
|
||||||
|
|
||||||
|
only the expected discord failures (already deleted, or no permission) are
|
||||||
|
swallowed; an unexpected error surfaces rather than being silently dropped.
|
||||||
|
"""
|
||||||
|
if not self.delete_message_after or self.message is None:
|
||||||
|
return
|
||||||
|
try:
|
||||||
|
await self.message.delete()
|
||||||
|
except (discord.NotFound, discord.Forbidden):
|
||||||
|
pass
|
||||||
|
|||||||
Reference in New Issue
Block a user