User "Brion VIBBER" posted a comment on MediaWiki.r86714.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/86714#c20277
Commit summary:

Allow extensions to customize the search forms. This required some cleanup and 
refactoring to special:search and search engine.
Should be fully backwards compatible. Lightly tested, but only with MySQL 
search backend.
Introduces concept of search profiles, which replace long list of namespaces in 
the url.

Comment:

First -- here in this commit comment would be nice.

Second -- in doc comments on the code would be nice.

Third -- whoever the audience who has to know what they are should be able to 
find that information. If we don't know what 'search profiles' are, we can't 
tell whether that's info that only people working on tweaking Special:Search 
have to worry about, or whether this is a concept that's being introduced to 
end-users that they'll have to select from and manage.

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to