Remove API 0.2

Author: frodrigo

apache-site

@@ -128,7 +128,6 @@
 #	RewriteCond %{REQUEST_URI} !/maintenance.html$
 #	RewriteRule !^maintenance.html$ maintenance.html [R=302,L]
 
-	RewriteRule "^(/api/0.2/.*)" "/en$1" [PT]
 	RewriteRule "^/([a-z_]+)(/api/0\.3/.*)" "$2?%{QUERY_STRING}&langs=$1" [PT]
 	RewriteRule "^/([a-z_]+)/error/(.*)" "/$1/issue/$2?%{QUERY_STRING}" [R,L]
 	RewriteRule "^/([a-z_]+)/errors/done(.*)" "/$1/issues/done$2" [R,L]

api/app.py

@@ -10,8 +10,7 @@
 
 openapi_tags = [
     {
-        "name": "0.2",
-        "description": "The 0.2 part of the API is deprecated.",
+        "name": "0.3",
     },
 ]
 

api/issue.py

@@ -19,43 +19,6 @@
 Status = Literal["done", "false"]
 
 
-async def _remove_bug_err_id(db: Connection, error_id: int, status: Status) -> int:
-    # find source
-    source_id = None
-    sql = "SELECT uuid,source_id,class FROM markers WHERE uuid_to_bigint(uuid) = $1"
-    for res in await db.fetch(sql, error_id):
-        uuid = res["uuid"]
-        source_id = res["source_id"]
-        class_id = res["class"]
-
-    if not source_id:
-        return -1
-
-    async with db.transaction():
-        await db.execute("DELETE FROM markers_status WHERE uuid=$1", uuid)
-
-        await db.execute(
-            """INSERT INTO markers_status
-                            (source_id,item,class,elems,date,status,lat,lon,subtitle,uuid)
-                        SELECT source_id,item,class,elems,NOW(),$1,
-                                lat,lon,subtitle,uuid
-                        FROM markers
-                        WHERE uuid = $2
-                        ON CONFLICT DO NOTHING""",
-            status,
-            uuid,
-        )
-
-        await db.execute("DELETE FROM markers WHERE uuid = $1", uuid)
-        await db.execute(
-            "UPDATE markers_counts SET count = count - 1 WHERE source_id = $1 AND class = $2",
-            source_id,
-            class_id,
-        )
-
-    return 0
-
-
 async def _remove_bug_uuid(db: Connection, uuid: UUID, status: Status) -> int:
     # find source
     source_id = None
@@ -162,19 +125,6 @@ def expand_tags(tags):
     return ret
 
 
-@router.get("/0.2/error/{err_id}", tags=["0.2"])
-async def error_err_id(
-    err_id: int, db: Connection = Depends(database.db)
-) -> Dict[str, Any]:
-    return _error(
-        2,
-        db,
-        ["en"],
-        None,
-        await _get(db, err_id=err_id),
-    )
-
-
 @router.get("/0.3/issue/{uuid}", tags=["issues"])
 async def error_uuid(
     uuid: UUID,
@@ -297,19 +247,6 @@ def _error(
         }
 
 
-@router.get("/0.2/error/{err_id}/{status}", tags=["0.2"])
-async def status_err_id(
-    request: Request,
-    err_id: int,
-    status: Status,
-    db: Connection = Depends(database.db_rw),
-) -> None:
-    if await _remove_bug_err_id(db, err_id, status) == 0:
-        return None
-    else:
-        raise HTTPException(status_code=410, detail="FAIL")
-
-
 @router.get("/0.3/issue/{uuid}/{status}", tags=["issues"])
 async def status_uuid(
     request: Request,

api/issues.py

@@ -1,5 +1,4 @@
 import json
-from collections import OrderedDict
 from itertools import groupby
 from typing import Any, Dict, List, Literal, Tuple
 
@@ -51,92 +50,6 @@ def render(self, content: Any) -> bytes:
         )
 
 
-@router.get("/0.2/errors", tags=["0.2"])
-async def errors(
-    request: Request,
-    db: Connection = Depends(database.db),
-    params=Depends(commons_params.params),
-) -> Dict[str, Any]:
-    results = await query._gets(db, params)
-    out: Dict[str, Any] = OrderedDict()
-
-    if not params.full:
-        out["description"] = ["lat", "lon", "error_id", "item"]
-    else:
-        out["description"] = [
-            "lat",
-            "lon",
-            "error_id",
-            "item",
-            "source",
-            "class",
-            "elems",
-            "subclass",
-            "subtitle",
-            "title",
-            "level",
-            "update",
-            "username",
-        ]
-    out["errors"] = []
-
-    for res in results:
-        lat = res["lat"]
-        lon = res["lon"]
-        error_id = res["id"]
-        item = res["item"] or 0
-
-        if not params.full:
-            out["errors"].append([str(lat), str(lon), str(error_id), str(item)])
-        else:
-            source = res["source_id"]
-            classs = res["class"]
-            elems = "_".join(
-                map(
-                    lambda elem: {"N": "node", "W": "way", "R": "relation"}[
-                        elem["type"]
-                    ]
-                    + str(elem["id"]),
-                    res["elems"] or [],
-                )
-            )
-            subclass = 0
-
-            subtitle_auto = utils.i10n_select(res["subtitle"], ["en"])
-            subtitle = subtitle_auto and subtitle_auto["auto"] or ""
-
-            title_auto = utils.i10n_select(res["title"], ["en"])
-            title = title_auto and title_auto["auto"] or ""
-
-            level = res["level"]
-            update = res["timestamp"]
-            username = ",".join(
-                map(
-                    lambda elem: "username" in elem and elem["username"] or "",
-                    res["elems"] or [],
-                )
-            )
-            out["errors"].append(
-                [
-                    str(lat),
-                    str(lon),
-                    str(error_id),
-                    str(item),
-                    str(source),
-                    str(classs),
-                    str(elems),
-                    str(subclass),
-                    subtitle,
-                    title,
-                    str(level),
-                    str(update),
-                    username,
-                ]
-            )
-
-    return out
-
-
 @router.get("/0.3/issues", tags=["issues"])
 @router.get("/0.3/issues.json", tags=["issues"])
 async def issues(