User talk:Legoktm/Memento
Latest comment: 10 years ago by Shawnmjones
I've been updating the code and committing. I haven't run the test suite yet.
Shall I update this page with my status so far?
Thanks again for the review.
Shawnmjones (talk) 02:04, 17 July 2014 (UTC)
- Yes please! :) Legoktm (talk) 03:10, 17 July 2014 (UTC)
I've got a few questions so far.
- For the onImageBeforeProduceHTML code:
- This code is experimental, and optional. It would be nice if it were enabled. I'm not sure how to solve the image problem for non-locally hosted files and would welcome any suggestions.
- For onArticleViewHeader code:
- getTitle()->exists() vs. getTitle()->isKnown():
- as noted in my comment, I was confused by the documentation
- getTitle()->exists() vs. getTitle()->isKnown():
For default configuration- I just made the values explicitly for the array to 1 through 15 rather than calculating it; not quite as flexible as I would like, but still a sensible default.- the $wgMementoExcludeNamespaces variable is supposed to be set, by default, to all of the non NS_MAIN namespaces; seeing as sites can configure their own namespaces, it was suggested by the wikitech-l folks to set this to array_diff( MWNamespace::getValidNamespaces(), MWNamespace::getContentNamespaces() ) by default
unfortunately, I cannot set this value using getValidNamespaces() at the entry point for the extension (Memento.php); the getValidNamespaces() blows up with "Unsupported operand types in /var/www/html/demo-devel/includes/Namespace.php on line 222"
- MementoResource->fetchMementoFromDatabase()
- I tried to use $db->selectRow(), but it returned null, so I set it back to $db->select() and it worked; I was very confused
- MementoResource->fixTemplate()
- I'm still not sure how to support the parser cache without disableCache()
- I haven't been able to test it yet, so I'm not sure selectRow() will work, see above comment
- TimeGateResourceFrom302TimeNegotiation
- I tried using OutputPage::redirect($url, 302), but it didn't redirect, so I went back to setting the headers; I was very confused
Again, thanks to your review, the code is much better than it was. Shawnmjones (talk) 03:27, 24 July 2014 (UTC)