[aur-dev] [PATCH] Store 10 significant digits of the popularity score
Even if we only display two digits after the decimal point, storing more digits internally allows us to order the search results more accurately. Signed-off-by: Lukas Fleischer <lfleischer@archlinux.org> --- schema/aur-schema.sql | 2 +- upgrading/4.0.0.txt | 2 +- web/template/pkg_search_results.php | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/schema/aur-schema.sql b/schema/aur-schema.sql index 9ac5d44..5a702b2 100644 --- a/schema/aur-schema.sql +++ b/schema/aur-schema.sql @@ -70,7 +70,7 @@ CREATE TABLE PackageBases ( ID INTEGER UNSIGNED NOT NULL AUTO_INCREMENT, Name VARCHAR(255) NOT NULL, NumVotes INTEGER UNSIGNED NOT NULL DEFAULT 0, - Popularity DECIMAL(6,2) UNSIGNED NOT NULL DEFAULT 0, + Popularity DECIMAL(10,6) UNSIGNED NOT NULL DEFAULT 0, OutOfDateTS BIGINT UNSIGNED NULL DEFAULT NULL, SubmittedTS BIGINT UNSIGNED NOT NULL, ModifiedTS BIGINT UNSIGNED NOT NULL, diff --git a/upgrading/4.0.0.txt b/upgrading/4.0.0.txt index fd04c44..954dfcc 100644 --- a/upgrading/4.0.0.txt +++ b/upgrading/4.0.0.txt @@ -35,7 +35,7 @@ CREATE TABLE PackageComaintainers ( ---- ALTER TABLE PackageBases - ADD COLUMN Popularity DECIMAL(6,2) UNSIGNED NOT NULL DEFAULT 0; + ADD COLUMN Popularity DECIMAL(10,6) UNSIGNED NOT NULL DEFAULT 0; ---- 6. Drop the category ID foreign key from the PackageBases table: diff --git a/web/template/pkg_search_results.php b/web/template/pkg_search_results.php index 5c8f73e..7134f2d 100644 --- a/web/template/pkg_search_results.php +++ b/web/template/pkg_search_results.php @@ -54,7 +54,7 @@ if (!$result): ?> <td><a href="<?= htmlspecialchars(get_pkg_uri($row["Name"]), ENT_QUOTES); ?>"><?= htmlspecialchars($row["Name"]) ?></a></td> <td<?php if ($row["OutOfDateTS"]): ?> class="flagged"<?php endif; ?>><?= htmlspecialchars($row["Version"]) ?></td> <td><?= $row["NumVotes"] ?></td> - <td><?= $row["Popularity"] ?></td> + <td><?= number_format($row["Popularity"], 2) ?></td> <?php if ($SID): ?> <td> <?php if (isset($row["Voted"])): ?> -- 2.4.3
participants (1)
-
Lukas Fleischer