Merge remote-tracking branch 'smcv/pagestats-show'

[git.ikiwiki.info.git] / doc / todo / calendar_autocreate.mdwn

diff --git a/doc/todo/calendar_autocreate.mdwn b/doc/todo/calendar_autocreate.mdwn
index 2a7350b79ef3656ac4f334d3415f6a8fdf4d378b..c1f9c454ec485e12253f955867ddc4ba4beecb66 100644 (file)
--- a/doc/todo/calendar_autocreate.mdwn
+++ b/doc/todo/calendar_autocreate.mdwn
@@ -1,5 +1,7 @@
 Here is a patch that makes [[ikiwiki-calendar]] almost useless.
 
 Here is a patch that makes [[ikiwiki-calendar]] almost useless.
 

+> [[merged|done]], thanks! --[[smcv]]
+

 It adds some options, the main one being `calendar_autocreate`, which is
 similar to the `tag_autocreate` option of the [[tag|plugins/tag]]: it create
 archive pages when needed.
 It adds some options, the main one being `calendar_autocreate`, which is
 similar to the `tag_autocreate` option of the [[tag|plugins/tag]]: it create
 archive pages when needed.


@@ -191,22 +193,30 @@ sub gencalendaryear {
 >
 > I think that should be `ikiwiki --setup ~/ikiwiki.setup --refresh`
 >
 >
 > I think that should be `ikiwiki --setup ~/ikiwiki.setup --refresh`
 >

+> > [[Corrected|https://github.com/paternal/ikiwiki/commit/213dad76d47bab9db8e44d6e20c8371960375e77]]
+>

 > The indentation of some of the new code in `IkiWiki/Plugin/calendar.pm`
 > is weird. Please use one hard tab (U+0009) per indent step: you seem
 > to have used a mixture of one hard tab per indent or two spaces
 > per indent, which looks bizarre for anyone whose tab size is not
 > 2 spaces.
 >
 > The indentation of some of the new code in `IkiWiki/Plugin/calendar.pm`
 > is weird. Please use one hard tab (U+0009) per indent step: you seem
 > to have used a mixture of one hard tab per indent or two spaces
 > per indent, which looks bizarre for anyone whose tab size is not
 > 2 spaces.
 >

+> > [[Corrected|https://github.com/paternal/ikiwiki/commit/1d97160dae775c31e166d9886472dacdd773d571]]
+>

 >     +        return unless $config{calendar_autocreate};
 >
 > This is checked in `gencalendaryear` but not in `gencalendarmonth`.
 > Shouldn't `gencalendarmonth` do it too? Alternatively, do the check
 > in `scan`, which calls `gencalendarmonth` directly.
 >
 >     +        return unless $config{calendar_autocreate};
 >
 > This is checked in `gencalendaryear` but not in `gencalendarmonth`.
 > Shouldn't `gencalendarmonth` do it too? Alternatively, do the check
 > in `scan`, which calls `gencalendarmonth` directly.
 >

+> > Once again, [[you are right|https://github.com/paternal/ikiwiki/commit/473bcbe7a42a4168cab82ed12185817248de045f]]
+>

 >     +                my $year  = $date[5] + 1900;
 >
 > You calculate this, but you don't seem to do anything with it?
 >
 >     +                my $year  = $date[5] + 1900;
 >
 > You calculate this, but you don't seem to do anything with it?
 >

+> > [[Corrected|https://github.com/paternal/ikiwiki/commit/d0b34951240317642543351ec62f98d3d8df8c0f]]
+>

 >     +  if (not exists $changed{$params{year}}) {
 >     +    $changed{$params{year}} = ();
 >     +  }
 >     +  if (not exists $changed{$params{year}}) {
 >     +    $changed{$params{year}} = ();
 >     +  }


@@ -222,4 +232,9 @@ sub gencalendaryear {
 > in order to put the pair `$params{month} => 1` in it (the term to look
 > up if you're curious is "autovivification").
 >
 > in order to put the pair `$params{month} => 1` in it (the term to look
 > up if you're curious is "autovivification").
 >

+> > [[Corrected|https://github.com/paternal/ikiwiki/commit/d0b34951240317642543351ec62f98d3d8df8c0f]]
+>

 > --[[smcv]]
 > --[[smcv]]

+>
+> > Thank you for your review.
+> > --[[Louis|spalax]]