[BUG] TypeError: strptime() argument 1 must be str, not datetime.date

[jgraichen]

Describe the bug

The YAML parser already converts time formats in the frontmatter into datetime objects. If reading from frontmatter is enabled, building the page fails with:

TypeError: strptime() argument 1 must be str, not datetime.datetime

To Reproduce
Steps to reproduce the behavior:

1. Configure mkdocs.yml:

   plugins:
     - blogging:
         meta_time_format: '%Y-%m-%d'
   

2. Create an article with one of the following metadata values:

   ---
   date: 2023-04-12
   ---
   

   ---
   date: 2023-04-12 12:12
   ---
   

Expected behavior

Building the page should not fail.

Otherwise, it would be nice if an available date and datetime would be taken without configuring meta_time_format, if it already is a valid date by YAML. This would make the blogging plugin compatible with mkdocs-rss-plugin.

Maybe something like this:

    def with_timestamp(self, page, by_revision):
        timestamp = self.get_datetime_from_meta(page)
        if "time" in page.meta:
            timestamp = self._parse_time(page.meta["time"])
        if "date" in page.meta and timestamp is None:
            timestamp = self._parse_time(page.meta["date"])
        # ...

    def _parse_time(self, value):
        if isinstance(value, date):
            return datetime.combine(value, datetime.min.time()).timestamp()
        if self.meta_time_format:
            return datetime.strptime(value, self.meta_time_format).timestamp()

Environment (please complete the following information):

• OS: Ubuntu 22.04
• Python version 3.10.6
• Plugin version 2.2.4

Additional context
Add any other context about the problem here.

[liang2kl]

Thank you for reporting this! Actually, I didn't know the parser will parse things into datetime objects, and it's a great idea to make use of that.

If you want a temporary fix, wrap the date with quotes to make it recognized as a string. I will try to implement the feature you requested as well.

[jgraichen]

Will it be ok to send a PR?

[liang2kl]

Will it be ok to send a PR?

Sure!