remove decorator because its stupid

Author: ZeroIntensity

CHANGELOG.md

@@ -13,7 +13,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Added the `templates` function
 - Added support for `attrs` in type validation
 - Added documentation for caching
-- Added `App.cache`
 - Added the `cache_rate` parameter to routers
 - Removed `psutil` and `plotext` as a global dependency
 - Added `fancy` optional dependencies

docs/building-projects/responses.md

@@ -18,50 +18,40 @@ async def index():
 
 Sometimes, computing the response for a route can be expensive or unnecessary. For this, view.py, along with many other web frameworks, provide the ability to cache responses.
 
-View lets you do this in one of two ways:
+View lets you do this by using the `cache_rate` parameter on a router.
 
-- Using the `cache` decorator (standard or direct).
-- Using the `cache_rate` parameter on a router.
-
-For example, with the `cache` decorator
+For example:
 
 ```py
-from view import new_app, cache
+from view import new_app
 
 app = new_app()
 
-@app.get("/")
-@cache(10)  # reload this route every 10 requests
+@app.get("/", cache_rate=10)  # reload this route every 10 requests
 async def index():
     return "..."
 
 app.run()
 ```
 
-Once again, a direct variation of `cache` is available to prevent an import:
-
-```py
-@app.get("/")
-@app.cache(10)  # equivalent to the above
-async def index():
-    return "..."
-```
-
-Finally, you may use the `cache_rate` parameter on any router:
+You can see this in more detail by using a route that changes it's responses:
 
 ```py
 from view import new_app
 
 app = new_app()
+count = 1
 
-@app.get("/", cache_rate=10)  # reload this route every 10 requests
+@app.get("/", cache_rate=10)
 async def index():
-    return "..."
+    global count
+    count += 1
+    return str(count)
 
 app.run()
 ```
 
-There is no difference between these two, and is simply up to personal preference. Note that the `cache` decorator will override the `cache_rate` parameter.
+In the above example, `index` is only called every 10 requests, so after 20 calls, `count` would be `2`.
 
 ## Response Protocol
 

src/view/app.py

@@ -43,7 +43,6 @@
 from .routing import body as body_impl
 from .routing import delete, get, options, patch, post, put
 from .routing import query as query_impl
-from .routing import cache as cache_impl
 from .typing import Callback, DocsType
 from .util import enable_debug
 
@@ -354,10 +353,6 @@ def put(self, path: str, doc: str | None = None, *, cache_rate: int = -1):
     def options(self, path: str, doc: str | None = None, *, cache_rate: int = -1):
         """Set a OPTIONS route."""
         return self._method_wrapper(path, doc, cache_rate, options)
-    
-    def cache(self, amount: int):
-        """Set the cache rate for a route. For example, if this is 10, the route will only be called every 10 requests."""
-        return cache_impl(amount)
 
     def _set_log_arg(self, kwargs: _LogArgs, key: str) -> None:
         if key not in kwargs:

src/view/routing.py

@@ -22,7 +22,6 @@
     "query",
     "body",
     "route_types",
-    "cache",
     "BodyParam",
 )
 
@@ -323,12 +322,3 @@ def inner(r: RouteOrCallable) -> Route:
 
     return inner
 
-
-def cache(amount: int):
-    """Set the cache rate for a route. For example, if this is 10, the route will only be called every 10 requests."""
-    def inner(r: RouteOrCallable) -> Route:
-        route = _ensure_route(r)
-        route.cache_rate = amount
-        return route
-
-    return inner

tests/test_app.py

@@ -5,7 +5,7 @@
 from typing_extensions import NotRequired
 from ward import test
 
-from view import BodyParam, Response, body, new_app, query, cache, get
+from view import BodyParam, Response, body, new_app, query, get
 
 
 @test("responses")
@@ -546,20 +546,6 @@ async def _():
     app = new_app()
     count = 0
 
-    @app.get("/")
-    @cache(10)
-    async def index():
-        nonlocal count
-        count += 1
-        return str(count)
-    
-    @app.get("/direct")
-    @app.cache(10)
-    async def direct():
-        nonlocal count
-        count += 1
-        return str(count)
-    
     @app.get("/param", cache_rate=10)
     async def param():
         nonlocal count
@@ -574,12 +560,6 @@ async def param_std():
 
 
     async with app.test() as test:
-        results = [(await test.get("/")).message for _ in range(10)]
-        assert all(i == results[0] for i in results)
-        
-        results = [(await test.get("/direct")).message for _ in range(10)]
-        assert all(i == results[0] for i in results)
-        
         results = [(await test.get("/param")).message for _ in range(10)]
         assert all(i == results[0] for i in results)