Added support for reordering flairs through reddit.flair.reorder

CHANGES.rst

@@ -6,6 +6,13 @@ PRAW follows `semantic versioning <http://semver.org/>`_.
 Unreleased
 ----------
 
+**Added**
+
+- :meth:`~.SubredditLinkFlairTemplates.reorder` to reorder a subreddit's link flair
+  templates.
+- :meth:`~.SubredditRedditorFlairTemplates.reorder` to reorder a subreddit's redditor
+  flair templates.
+
 7.7.1 (2023/07/11)
 ------------------
 

praw/endpoints.py

@@ -63,6 +63,7 @@
     "flairtemplate_v2":        "r/{subreddit}/api/flairtemplate_v2",
     "flairtemplateclear":      "r/{subreddit}/api/clearflairtemplates/",
     "flairtemplatedelete":     "r/{subreddit}/api/deleteflairtemplate/",
+    "flairtemplatereorder":    "r/{subreddit}/api/flair_template_order",
     "friend":                  "r/{subreddit}/api/friend/",
     "friend_v1":               "api/v1/me/friends/{user}",
     "friends":                 "api/v1/me/friends/",

praw/models/reddit/subreddit.py

@@ -830,6 +830,11 @@ def _fetch(self):
         self.__dict__.update(other.__dict__)
         self._fetched = True
 
+    def _fetch_data(self) -> dict:
+        name, fields, params = self._fetch_info()
+        path = API_PATH[name].format(**fields)
+        return self._reddit.request(method="GET", params=params, path=path)
+
     def _fetch_info(self):
         return "subreddit_about", {"subreddit": self}, None
 
@@ -2225,6 +2230,17 @@ def delete(self, template_id: str):
         url = API_PATH["flairtemplatedelete"].format(subreddit=self.subreddit)
         self.subreddit._reddit.post(url, data={"flair_template_id": template_id})
 
+    def _reorder(self, flair_list: list, *, is_link: Optional[bool] = None):
+        url = API_PATH["flairtemplatereorder"].format(subreddit=self.subreddit)
+        self.subreddit._reddit.patch(
+            url,
+            params={
+                "flair_type": self.flair_type(is_link),
+                "subreddit": self.subreddit.display_name,
+            },
+            json=flair_list,
+        )
+
     @_deprecate_args(
         "template_id",
         "text",
@@ -2728,7 +2744,7 @@ def update(
         :param submit_text_label: Custom label for submit text post button (``None`` for
             default).
         :param subreddit_type: One of ``"archived"``, ``"employees_only"``,
-            ``"gold_only"``, ``"gold_restricted"``, ``"private"``, ``"public"``, or
+            ``"gold_only"``, ``gold_restricted``, ``"private"``, ``"public"``, or
             ``"restricted"``.
         :param suggested_comment_sort: All comment threads will use this sorting method
             by default. Leave ``None``, or choose one of ``"confidence"``,
@@ -4130,6 +4146,22 @@ def clear(self):
         """
         self._clear(is_link=True)
 
+    def reorder(self, flair_list: List[str]):
+        """Reorder a list of flairs.
+
+        :param flair_list: A list of flair IDs.
+
+        For example, to reverse the order of the link flair list try:
+
+        .. code-block:: python
+
+            subreddit = reddit.subreddit("test")
+            flairs = [flair["id"] for flair in subreddit.flair.link_templates]
+            subreddit.flair.link_templates.reorder(list(reversed(flairs)))
+
+        """
+        self._reorder(flair_list, is_link=True)
+
     def user_selectable(
         self,
     ) -> Generator[Dict[str, Union[str, bool]], None, None]:
@@ -4245,3 +4277,20 @@ def clear(self):
 
         """
         self._clear(is_link=False)
+
+    def reorder(self, flair_list: List[str]):
+        """Reorder a list of flairs.
+
+        :param flair_list: A list of flair IDs.
+
+        For example, to reverse the order of the :class:`.Redditor` flair templates list
+        try:
+
+        .. code-block:: python
+
+            subreddit = reddit.subreddit("test")
+            flairs = [flair["id"] for flair in subreddit.flair.templates]
+            subreddit.flair.templates.reorder(list(reversed(flairs)))
+
+        """
+        self._reorder(flair_list, is_link=False)

praw/reddit.py

@@ -13,6 +13,7 @@
     Dict,
     Generator,
     Iterable,
+    List,
     Optional,
     Type,
     Union,
@@ -493,7 +494,7 @@ def _objectify_request(
         *,
         data: Optional[Union[Dict[str, Union[str, Any]], bytes, IO, str]] = None,
         files: Optional[Dict[str, IO]] = None,
-        json: Optional[Dict[Any, Any]] = None,
+        json: Optional[Union[Dict[Any, Any], List[Any]]] = None,
         method: str = "",
         params: Optional[Union[str, Dict[str, str]]] = None,
         path: str = "",
@@ -670,7 +671,7 @@ def delete(
         path: str,
         *,
         data: Optional[Union[Dict[str, Union[str, Any]], bytes, IO, str]] = None,
-        json: Optional[Dict[Any, Any]] = None,
+        json: Optional[Union[Dict[Any, Any], List[Any]]] = None,
         params: Optional[Union[str, Dict[str, str]]] = None,
     ) -> Any:
         """Return parsed objects returned from a DELETE request to ``path``.
@@ -794,7 +795,8 @@ def patch(
         path: str,
         *,
         data: Optional[Union[Dict[str, Union[str, Any]], bytes, IO, str]] = None,
-        json: Optional[Dict[Any, Any]] = None,
+        json: Optional[Union[Dict[Any, Any], List[Any]]] = None,
+        params: Optional[Union[str, Dict[str, str]]] = None,
     ) -> Any:
         """Return parsed objects returned from a PATCH request to ``path``.
 
@@ -804,9 +806,12 @@ def patch(
         :param json: JSON-serializable object to send in the body of the request with a
             Content-Type header of application/json (default: ``None``). If ``json`` is
             provided, ``data`` should not be.
+        :param params: The query parameters to add to the request (default: ``None``).
 
         """
-        return self._objectify_request(data=data, json=json, method="PATCH", path=path)
+        return self._objectify_request(
+            data=data, json=json, method="PATCH", params=params, path=path
+        )
 
     @_deprecate_args("path", "data", "files", "params", "json")
     def post(
@@ -815,7 +820,7 @@ def post(
         *,
         data: Optional[Union[Dict[str, Union[str, Any]], bytes, IO, str]] = None,
         files: Optional[Dict[str, IO]] = None,
-        json: Optional[Dict[Any, Any]] = None,
+        json: Optional[Union[Dict[Any, Any], List[Any]]] = None,
         params: Optional[Union[str, Dict[str, str]]] = None,
     ) -> Any:
         """Return parsed objects returned from a POST request to ``path``.
@@ -863,7 +868,7 @@ def put(
         path: str,
         *,
         data: Optional[Union[Dict[str, Union[str, Any]], bytes, IO, str]] = None,
-        json: Optional[Dict[Any, Any]] = None,
+        json: Optional[Union[Dict[Any, Any], List[Any]]] = None,
     ):
         """Return parsed objects returned from a PUT request to ``path``.
 
@@ -913,7 +918,7 @@ def request(
         *,
         data: Optional[Union[Dict[str, Union[str, Any]], bytes, IO, str]] = None,
         files: Optional[Dict[str, IO]] = None,
-        json: Optional[Dict[Any, Any]] = None,
+        json: Optional[Union[Dict[Any, Any], List[Any]]] = None,
         method: str,
         params: Optional[Union[str, Dict[str, Union[str, int]]]] = None,
         path: str,