Skip to content

Commit 08e28db

Browse files
committed
Use distvname instead of dist not to check distributions with different versions
1 parent b6954ee commit 08e28db

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

lib/pause_2017/PAUSE/Web/Controller/User/Files.pm

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -176,11 +176,11 @@ sub delete {
176176
my $tmpf = $f;
177177
$tmpf =~ s/\.(?:readme|meta)$/.tar.gz/;
178178
my $info = CPAN::DistnameInfo->new($tmpf);
179-
my $dist = $info->dist;
179+
my $distv = $info->distvname;
180180
my $blurb = $deletes{$f} ?
181181
$c->scheduled($whendele{$f}) :
182182
HTTP::Date::time2str((stat _)[9]);
183-
$files{$f} = {stat => -s _, blurb => $blurb, indexed => $indexed->{$f}, dist => $dist };
183+
$files{$f} = {stat => -s _, blurb => $blurb, indexed => $indexed->{$f}, distv => $distv };
184184
$pause->{deleting_indexed_files} = 1 if $deletes{$f} && $indexed->{$f};
185185
}
186186
$pause->{files} = \%files;

lib/pause_2017/templates/user/files/delete.html.ep

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
<tbody class="list">
3030
% for my $file (sort keys %$files) {
3131
<tr>
32-
<td class="checkbox"><%= check_box "pause99_delete_files_FILE" => $file, 'data-dist' => $files->{$file}{dist} %></td>
32+
<td class="checkbox"><%= check_box "pause99_delete_files_FILE" => $file, 'data-distv' => $files->{$file}{distv} %></td>
3333
% if ($files->{$file}{indexed}) {
3434
<td class="file indexed"><%= $file %> [indexed]</td>
3535
% } else {
@@ -54,8 +54,8 @@ var List = new List('files', {
5454
document.querySelectorAll('input[type=checkbox]').forEach(function(e) {
5555
e.addEventListener('change', function(ev) {
5656
var checked = ev.currentTarget.checked;
57-
var dist = ev.currentTarget.getAttribute('data-dist');
58-
document.querySelectorAll('input[data-dist='+dist+']').forEach(function(e) {
57+
var distv = ev.currentTarget.getAttribute('data-distv');
58+
document.querySelectorAll('input[data-distv="'+distv+'"]').forEach(function(e) {
5959
e.checked = checked;
6060
});
6161
})

0 commit comments

Comments
 (0)