Remove extra lookup

Author: lpil

src/packages/router.gleam

@@ -154,10 +154,8 @@ fn search(request: Request, context: Context) -> Response {
       })
     _ -> packages
   }
-  let assert Ok(total_package_count) =
-    storage.get_total_package_count(context.db)
 
-  page.packages_list(packages, total_package_count, search_term)
+  page.packages_list(packages, search_term)
   |> wisp.html_response(200)
 }
 

src/packages/storage.gleam

@@ -321,13 +321,6 @@ pub fn get_optional_package(
   |> result.map_error(error.StorageError)
 }
 
-pub fn get_total_package_count(database: Database) -> Result(Int, Error) {
-  database.packages
-  |> storail.list([])
-  |> result.map(list.length)
-  |> result.map_error(error.StorageError)
-}
-
 pub fn upsert_release(
   database: Database,
   package: String,

src/packages/web/page.gleam

@@ -11,14 +11,10 @@ import lustre/element/html
 import packages/storage.{type Package}
 import packages/web/icons
 
-pub fn packages_list(
-  packages: List(Package),
-  total_package_count: Int,
-  search_term: String,
-) -> String {
+pub fn packages_list(packages: List(Package), search_term: String) -> String {
   html.div(
     [attribute.class("content")],
-    search_aware_package_list(packages, total_package_count, search_term),
+    search_aware_package_list(packages, search_term),
   )
   |> layout
 }
@@ -105,13 +101,12 @@ fn search_form(search_term: String) -> Element(Nil) {
 
 fn search_aware_package_list(
   packages: List(Package),
-  total_package_count: Int,
   search_term: String,
 ) -> List(Element(Nil)) {
   let header_phrase = case search_term, packages {
     "", [] -> "No packages have been added yet"
     "", [_] -> "1 package is available!"
-    "", _ -> int.to_string(total_package_count) <> " packages are available!"
+    "", _ -> int.to_string(list.length(packages)) <> " packages are available!"
 
     _, [] -> "No packages match your query"
     _, [_] -> "1 package matches your query!"