Use Mojo::Pg automatic json decoding instead of doing it manually
authorVincent Tondellier <tonton+hg@team1664.org>
Sun, 27 Nov 2016 17:17:43 +0100
changeset 119 0a9171619fd3
parent 118 a8ff5a8a4fe8
child 120 1a0fa98037fa
Use Mojo::Pg automatic json decoding instead of doing it manually
lib/CrashTest/Helper/BugLinks.pm
lib/CrashTest/Plugin/Storage/Sql/Model/CrashGroup.pm
lib/CrashTest/Plugin/Storage/Sql/Model/CrashReport.pm
--- a/lib/CrashTest/Helper/BugLinks.pm	Sat Nov 26 18:33:57 2016 +0100
+++ b/lib/CrashTest/Helper/BugLinks.pm	Sun Nov 27 17:17:43 2016 +0100
@@ -13,8 +13,6 @@
 
 package CrashTest::Helper::BugLinks;
 use Mojo::Base 'Mojolicious::Plugin';
-use Mojo::JSON qw/decode_json/;
-use Mojo::Util qw/dumper/;
 
 has [ qw/config app/ ];
 
@@ -55,12 +53,10 @@
 
 
 sub _bug_links {
-    my ($self, $c, $json) = @_;
+    my ($self, $c, $bugs) = @_;
 
-    #return dumper $bugs;
-    return "" if(!defined($json) || $json eq "");
+    return "" if(!defined($bugs));
 
-    my $bugs = decode_json($json);
     my @links;
 
     foreach my $bug(@$bugs) {
--- a/lib/CrashTest/Plugin/Storage/Sql/Model/CrashGroup.pm	Sat Nov 26 18:33:57 2016 +0100
+++ b/lib/CrashTest/Plugin/Storage/Sql/Model/CrashGroup.pm	Sun Nov 27 17:17:43 2016 +0100
@@ -100,7 +100,7 @@
         FETCH NEXT (?) ROWS ONLY
         ",
         @values, $pager->skipped, $pager->entries_per_page
-    )->hashes;
+    )->expand->hashes;
 
     return ($results, $pager);
 }
--- a/lib/CrashTest/Plugin/Storage/Sql/Model/CrashReport.pm	Sat Nov 26 18:33:57 2016 +0100
+++ b/lib/CrashTest/Plugin/Storage/Sql/Model/CrashReport.pm	Sun Nov 27 17:17:43 2016 +0100
@@ -99,7 +99,7 @@
         ",
         @values,
         $pager->skipped, $pager->entries_per_page
-    )->hashes;
+    )->expand->hashes;
 
     return ($results, $pager);
 }