diff options
author | Andinus <andinus@nand.sh> | 2022-06-11 15:49:16 +0530 |
---|---|---|
committer | Andinus <andinus@nand.sh> | 2022-06-11 15:49:16 +0530 |
commit | 79aa4269eccfc6afdb6347b37956b16e7603f0b0 (patch) | |
tree | da5dfefba6c53336df25caf66a91406d329c451f /lib/Crater/Gallery.rakumod | |
parent | 74d75501d6ac26e84f07d25a04a4639bcf1d5f61 (diff) | |
download | crater-79aa4269eccfc6afdb6347b37956b16e7603f0b0.tar.gz |
Handle sub-directories, show error on 0 images
Diffstat (limited to 'lib/Crater/Gallery.rakumod')
-rw-r--r-- | lib/Crater/Gallery.rakumod | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/lib/Crater/Gallery.rakumod b/lib/Crater/Gallery.rakumod index 448593f..373231a 100644 --- a/lib/Crater/Gallery.rakumod +++ b/lib/Crater/Gallery.rakumod @@ -10,9 +10,17 @@ class Crater::Gallery { #| Accessor for $!title. method title() { $!title } - method list() { + method list(:@sub-dir) { + # This will be considered an attempt to attack. There is no + # reason to check '.' I belive. + if @sub-dir.grep('.'|'..').elems { + die "[!!!] @sub-dir contains '..'/'.'"; + } + my @gallery; - my @paths = $!directory.dir; + my @paths = @sub-dir + ?? $!directory.add(@sub-dir.join("/")).dir + !! $!directory.dir; with $!title { push @gallery, %( :type<heading>, :text($_) ); @@ -20,6 +28,7 @@ class Crater::Gallery { # Add directories on top. for @paths.grep(*.d) { + next if .ends-with(".crater"); push @gallery, %( :type<directory>, :text($_.relative($!directory)) ); } |