external/aurweb
0
0
Fork 0
mirror of https://gitlab.archlinux.org/archlinux/aurweb.git synced 2025-02-03 10:43:03 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Explicitly initialize PackageBases.FlaggerComment

Browse source 
Since commit 09cb61a (schema: Remove invalid default values for TEXT
columns, 2017-04-15), the PackageBases.FlaggerComment field no longer
has a default value. Initialize this field explicitly whenever a new row
is added to the PackageBases table.

Signed-off-by: Lukas Fleischer <lfleischer@archlinux.org>
This commit is contained in:
Lukas Fleischer 2017-04-24 21:16:04 +02:00
parent 90ab7c7b83
commit f4e2d143ad
6 changed files with 25 additions and 23 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
aurweb/git/serve.py
View file

@ -65,8 +65,9 @@ def create_pkgbase(pkgbase, user):
now = int(time.time()) now = int(time.time())
cur = conn.execute("INSERT INTO PackageBases (Name, SubmittedTS, " + cur = conn.execute("INSERT INTO PackageBases (Name, SubmittedTS, " +
"ModifiedTS, SubmitterUID, MaintainerUID) VALUES " + "ModifiedTS, SubmitterUID, MaintainerUID, " +
"(?, ?, ?, ?, ?)", [pkgbase, now, now, userid, userid]) "FlaggerComment) VALUES (?, ?, ?, ?, ?, '')",
[pkgbase, now, now, userid, userid])
pkgbase_id = cur.lastrowid pkgbase_id = cur.lastrowid
cur = conn.execute("INSERT INTO PackageNotifications " + cur = conn.execute("INSERT INTO PackageNotifications " +

5
aurweb/git/update.py
View file

@ -61,8 +61,9 @@ def create_pkgbase(conn, pkgbase, user):
now = int(time.time()) now = int(time.time())
cur = conn.execute("INSERT INTO PackageBases (Name, SubmittedTS, " + cur = conn.execute("INSERT INTO PackageBases (Name, SubmittedTS, " +
"ModifiedTS, SubmitterUID, MaintainerUID) VALUES " + "ModifiedTS, SubmitterUID, MaintainerUID, " +
"(?, ?, ?, ?, ?)", [pkgbase, now, now, userid, userid]) "FlaggerComment) VALUES (?, ?, ?, ?, ?, '')",
[pkgbase, now, now, userid, userid])
pkgbase_id = cur.lastrowid pkgbase_id = cur.lastrowid
cur = conn.execute("INSERT INTO PackageNotifications " + cur = conn.execute("INSERT INTO PackageNotifications " +

14
test/t1300-git-update.sh
View file

@ -21,7 +21,7 @@ test_expect_success 'Test update hook on a fresh repository.' '
1|1|foobar|1-1|aurweb test package.|https://aur.archlinux.org/ 1|1|foobar|1-1|aurweb test package.|https://aur.archlinux.org/
1|GPL 1|GPL
1|1 1|1
1|1|python-pygit2|| 1|1|python-pygit2|||
1|1 1|1
EOF EOF
dump_package_info >actual && dump_package_info >actual &&
@ -42,8 +42,8 @@ test_expect_success 'Test update hook on another fresh repository.' '
2|MIT 2|MIT
1|1 1|1
2|2 2|2
1|1|python-pygit2|| 1|1|python-pygit2|||
2|1|python-pygit2|| 2|1|python-pygit2|||
1|1 1|1
2|1 2|1
EOF EOF
@ -63,8 +63,8 @@ test_expect_success 'Test update hook on an updated repository.' '
2|MIT 2|MIT
2|2 2|2
3|1 3|1
2|1|python-pygit2|| 2|1|python-pygit2|||
3|1|python-pygit2|| 3|1|python-pygit2|||
1|1 1|1
2|1 2|1
EOF EOF
@ -82,8 +82,8 @@ test_expect_success 'Test restore mode.' '
2|MIT 2|MIT
2|2 2|2
3|1 3|1
2|1|python-pygit2|| 2|1|python-pygit2|||
3|1|python-pygit2|| 3|1|python-pygit2|||
1|1 1|1
2|1 2|1
EOF EOF

8
test/t2100-mkpkglists.sh
View file

@ -14,10 +14,10 @@ test_expect_success 'Test package list generation with no packages.' '
test_expect_success 'Test package list generation.' ' test_expect_success 'Test package list generation.' '
cat <<-EOD | sqlite3 aur.db && cat <<-EOD | sqlite3 aur.db &&
INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS) VALUES (1, "foobar", 1, 0, 0); INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (1, "foobar", 1, 0, 0, "");
INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS) VALUES (2, "foobar2", 2, 0, 0); INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (2, "foobar2", 2, 0, 0, "");
INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS) VALUES (3, "foobar3", NULL, 0, 0); INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (3, "foobar3", NULL, 0, 0, "");
INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS) VALUES (4, "foobar4", 1, 0, 0); INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (4, "foobar4", 1, 0, 0, "");
INSERT INTO Packages (ID, PackageBaseID, Name) VALUES (1, 1, "pkg1"); INSERT INTO Packages (ID, PackageBaseID, Name) VALUES (1, 1, "pkg1");
INSERT INTO Packages (ID, PackageBaseID, Name) VALUES (2, 1, "pkg2"); INSERT INTO Packages (ID, PackageBaseID, Name) VALUES (2, 1, "pkg2");
INSERT INTO Packages (ID, PackageBaseID, Name) VALUES (3, 1, "pkg3"); INSERT INTO Packages (ID, PackageBaseID, Name) VALUES (3, 1, "pkg3");

8
test/t2300-pkgmaint.sh
View file

@ -8,10 +8,10 @@ test_expect_success 'Test package base cleanup script.' '
now=$(date -d now +%s) && now=$(date -d now +%s) &&
threedaysago=$(date -d "3 days ago" +%s) && threedaysago=$(date -d "3 days ago" +%s) &&
cat <<-EOD | sqlite3 aur.db && cat <<-EOD | sqlite3 aur.db &&
INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS) VALUES (1, "foobar", 1, $now, 0); INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (1, "foobar", 1, $now, 0, "");
INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS) VALUES (2, "foobar2", 2, $threedaysago, 0); INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (2, "foobar2", 2, $threedaysago, 0, "");
INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS) VALUES (3, "foobar3", NULL, $now, 0); INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (3, "foobar3", NULL, $now, 0, "");
INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS) VALUES (4, "foobar4", NULL, $threedaysago, 0); INSERT INTO PackageBases (ID, Name, PackagerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (4, "foobar4", NULL, $threedaysago, 0, "");
EOD EOD
"$PKGMAINT" && "$PKGMAINT" &&
cat <<-EOD >expected && cat <<-EOD >expected &&

8
test/t2500-notify.sh
View file

@ -6,10 +6,10 @@ test_description='notify tests'
test_expect_success 'Test out-of-date notifications.' ' test_expect_success 'Test out-of-date notifications.' '
cat <<-EOD | sqlite3 aur.db && cat <<-EOD | sqlite3 aur.db &&
INSERT INTO PackageBases (ID, Name, MaintainerUID, SubmittedTS, ModifiedTS) VALUES (1, "foobar", 1, 0, 0); INSERT INTO PackageBases (ID, Name, MaintainerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (1, "foobar", 1, 0, 0, "");
INSERT INTO PackageBases (ID, Name, MaintainerUID, SubmittedTS, ModifiedTS) VALUES (2, "foobar2", 2, 0, 0); INSERT INTO PackageBases (ID, Name, MaintainerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (2, "foobar2", 2, 0, 0, "");
INSERT INTO PackageBases (ID, Name, MaintainerUID, SubmittedTS, ModifiedTS) VALUES (3, "foobar3", NULL, 0, 0); INSERT INTO PackageBases (ID, Name, MaintainerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (3, "foobar3", NULL, 0, 0, "");
INSERT INTO PackageBases (ID, Name, MaintainerUID, SubmittedTS, ModifiedTS) VALUES (4, "foobar4", 1, 0, 0); INSERT INTO PackageBases (ID, Name, MaintainerUID, SubmittedTS, ModifiedTS, FlaggerComment) VALUES (4, "foobar4", 1, 0, 0, "");
INSERT INTO PackageComaintainers (PackageBaseID, UsersID, Priority) VALUES (1, 2, 1); INSERT INTO PackageComaintainers (PackageBaseID, UsersID, Priority) VALUES (1, 2, 1);
INSERT INTO PackageComaintainers (PackageBaseID, UsersID, Priority) VALUES (1, 4, 2); INSERT INTO PackageComaintainers (PackageBaseID, UsersID, Priority) VALUES (1, 4, 2);
INSERT INTO PackageComaintainers (PackageBaseID, UsersID, Priority) VALUES (2, 3, 1); INSERT INTO PackageComaintainers (PackageBaseID, UsersID, Priority) VALUES (2, 3, 1);