X-Git-Url: http://git.shiar.nl/sheet.git/blobdiff_plain/e69b3e72053f8594f4194520869cb0b8b6aed1b4..77eb26995a1185e50b2b275f058264857bed9cc7:/writer.plp?ds=inline

diff --git a/writer.plp b/writer.plp
index 73f6b98..275909b 100644
--- a/writer.plp
+++ b/writer.plp
@@ -108,67 +108,88 @@ my $db = eval {
 } or Abort('Database error', 501, $@);
 
 my @wordcols = (
+	lang    => 'Language',
+	cat     => 'Category',
 	form    => 'Translation',
 	wptitle => 'Wikipedia',
-	ref     => 'Reference',
-	cat     => 'Category',
-	lang    => 'Language',
 	source  => 'Image URL',
 	thumb   => 'Convert options',
+	prio    => 'Level',
+	ref     => 'Reference',
 );
 my ($find) = map {{id => $_}} $fields{id} || $Request || ();
 
 my $row;
+if ($find) {
+	$row = $db->select(word => '*', $find)->hash
+		or Abort("Word not found", 404);
+}
+
 if (exists $get{copy}) {
-	$row = {%fields{ qw(lang cat) }};
+	$row = {%{$row}{ qw(prio lang cat) }};
 }
-elsif ($ENV{REQUEST_METHOD} eq 'POST') {
+elsif ($ENV{REQUEST_METHOD} eq 'POST') {{
+	my $replace = $row;
 	$row = {%post{ pairkeys @wordcols }};
 	$_ = length ? $_ : undef for values %{$row};
+
 	eval {
 		my %res = (returning => '*');
 		my $query = $find ? $db->update(word => $row, $find, \%res) :
 			$db->insert(word => $row, \%res);
 		$row = $query->hash;
-	} or Alert("Entry could not be saved", $@);
+	} or do {
+		Alert("Entry could not be saved", $@);
+		next;
+	};
 
 	my $imgpath = "data/word/org/$row->{id}.jpg";
-	if (my $download = $row->{source} and !-e $imgpath) {
-		require LWP::UserAgent;
-		my $ua = LWP::UserAgent->new;
-		$ua->agent('/');
-		my $status = $ua->mirror($download, $imgpath);
-		$status->is_success or Alert([
-			"Source image not found",
-			"Download from <q>$download</q> failed: ".$status->status_line,
-		]);
+	if (($row->{source} // '') ne ($replace->{source} // '')) {
+		# copy changed remote url to local file
+		unlink $imgpath if -e $imgpath;
+		if (my $download = $row->{source}) {
+			require LWP::UserAgent;
+			my $ua = LWP::UserAgent->new;
+			$ua->agent('/');
+			my $status = $ua->mirror($download, $imgpath);
+			$status->is_success or Alert([
+				"Source image not found",
+				"Download from <q>$download</q> failed: ".$status->status_line,
+			]);
+		}
+	}
+	elsif ($row->{thumb} ~~ $replace->{thumb}) {
+		# image and conversion unaltered
+		$imgpath = undef;
 	}
 
 	my $thumbpath = "data/word/eng/$row->{form}.jpg";
-	if (-e $imgpath) {
-		my @cmds = @{ $row->{thumb} // [] };
-		@cmds = (
-			'convert',
-			-delete => '1--1', -background => 'white',
-			-gravity => @cmds ? 'northwest' : 'center',
-			@cmds,
-			-resize => '300x200^', -extent => '300x200',
-			'-strip', -quality => '60%', -interlace => 'plane',
-			$imgpath => $thumbpath
-		);
-		my $status = system @cmds;
-		$status == 0 or Alert([
-			"Thumbnail image not generated",
-			"Failed to convert source image, error code ".($status >> 8),
-		], "@cmds");
+	if ($imgpath) {
+		if (-e $imgpath) {
+			my @cmds = @{ $row->{thumb} // [] };
+			@cmds = (
+				'convert',
+				-delete => '1--1', -background => 'white',
+				-gravity => @cmds ? 'northwest' : 'center',
+				@cmds,
+				-resize => '300x200^', -extent => '300x200',
+				'-strip', -quality => '60%', -interlace => 'plane',
+				$imgpath => $thumbpath
+			);
+			my $status = system @cmds;
+			$status == 0 or Alert([
+				"Thumbnail image not generated",
+				"Failed to convert source image, error code ".($status >> 8),
+			], "@cmds");
+		}
+		else {
+			unlink $thumbpath;
+		}
 	}
-}
-elsif ($find) {
-	$row = $db->select(word => '*', $find)->hash
-		or Abort("Word not found", 404);
-}
+}}
 else {
-	$row = \%fields;
+	$row->{prio} //= 1;
+	$row->{$_} = $get{$_} for keys %get;
 }
 
 my $title = $row->{id} ? "entry <small>#$row->{id}</small>" : 'new entry';