Change Details

WIP This task is a bit of a more advanced task. It is recommended you have already done at least two tasks involving PHP coding of MediaWiki extensions that involved submitting code to gerrit. The DynamicPageList_(Wikimedia) extension (also known as the "intersection" extension) https://www.mediawiki.org/wiki/Extension:DynamicPageList_(Wikimedia) has an option `mode=gallery` which allows showing a gallery of images that match the query. Currently this only really works with files in the file namespace. Your task is to have it use the page image (from the PageImages extension https://www.mediawiki.org/wiki/Extension:PageImages ) for pages that are not file pages. To do this, you need to modify two parts: Where the SQL is generated (around line 441-ish of intersection/includes/DynamicPageListHooks.php ), provided you are in gallery mode, and the PageImages extension is loaded, you would have to add page_props to the $tables. You should use an alias so the key to the array is the alias name, and the value is `page_props`. You then need to add a join to $join for page_props. it should be a LEFT JOIN and the ON condition should be pp_propname => PageImages::PROP_NAME. You should then do the same thing with a different alias but for PageImages::PROP_NAME_FREE. You then have to add aliasforpageprops.pp_value (Probably also giving this an alias for clarify) to $fields You then have to find the code that calls $gallery->add. Instead, if $title->getNamespace() !== NS_FILE you have to get the field from the page_prop (trying first the free version, and then not free version if free is null), create a title from it (use Title::makeTitleSafe), and add that instead of the current title. However if you are not adding the current title, you also have to make sure the link goes to the right place, so you have to provide the fourth argument to $gallery->add as a link to the original title, e.g. $title->getLinkURL()

This task is a bit of a more advanced task. It is recommended you have already done at least two (maybe more) tasks involving PHP coding of MediaWiki extensions that involved submitting code to gerrit. As this is meant as a more challenging task, some of the details are not as spelled out as some of the less challenging tasks. If things are confusing, don't hesitate to ask for clarification on irc. The DynamicPageList_(Wikimedia) extension (also known as the "intersection" extension) https://www.mediawiki.org/wiki/Extension:DynamicPageList_(Wikimedia) has an option `mode=gallery` which allows showing a gallery of images that match the query. Currently this only really works with files in the file namespace. Your task is to have it use the page image (from the PageImages extension https://www.mediawiki.org/wiki/Extension:PageImages ) for pages that are not file pages. Currently it just displays an empty space for pages not in the file namespace. Prerequisites: * You have done several MediaWiki PHP coding related tasks, and are looking for a more complex PHP coding task * You have already setup MediaWiki locally * You have a basic knowledge of SQL and know what a LEFT JOIN is (e.g. You understand what the query `SELECT page_namespace, page_title, p1.pp_value AS 'image' FROM page left join page_props p1 on p1.pp_page = page_id and p1.pp_propname = 'page_image_free' where page_id = 1931416;` means. If not, there are several sql tutorials online) It is recommended you skim through https://www.mediawiki.org/wiki/Manual:Page_props_table and https://www.mediawiki.org/wiki/Manual:Database_access#Database_Abstraction_Layer to give you background as to how MediaWiki makes SQL queries, if you are not already familiar with this. However that page goes in a lot of detail, so don't worry if you don't understand all of it. To do this, you need to modify two parts of the dynamic page list extension: Where the SQL is generated (around line 441-ish of intersection/includes/DynamicPageListHooks.php ), provided you are in gallery mode, and the PageImages extension is loaded, you would have to add page_props to the $tables. You should use an alias so the key to the array is the alias name, and the value is `page_props`. You then need to add a join to $join for page_props. it should be a LEFT JOIN and the two ON conditions should be pp_propname = PageImages::PROP_NAME and page_id = pp_page (adding appropriate table aliases). You should then do the same thing with a different alias but for PageImages::PROP_NAME_FREE. You then have to add <alias>.pp_value to $fields (Probably also giving this an alias for clarify). Doing these modifications show change the query so that the database returns a column containing the "free" page image and a column containing the "non-free" page image Once the query is modified, we now have to make use of the modified query. Now you must adjust the gallery building code to use the alternative images for pages that are not in the file namespace. You then have to find the code that calls $gallery->add. Instead of the current code, if $title->getNamespace() !== NS_FILE you have to get the field from the page_prop (trying first the free version, and then not free version if free is null), create a title from it (use Title::makeTitleSafe), and add that instead of the current title. However if you are not adding the current title, you also have to make sure the link goes to the right place, so you have to provide the fourth argument to $gallery->add as a link to the original title, e.g. $title->getLinkURL() Once all this is done, test the code on your local wiki to make sure it all works. Make a commit with an appropriate commit message, and upload to gerrit.

WIPThis task is a bit of a more advanced task. It is recommended you have already done at least two (maybe more) tasks involving PHP coding of MediaWiki extensions that involved submitting code to gerrit. As this is meant as a more challenging task, some of the details are not as spelled out as some of the less challenging tasks. If things are confusing, don't hesitate to ask for clarification on irc. This task is a bit of a more advanced taske DynamicPageList_(Wikimedia) extension (also known as the "intersection" extension) https://www.mediawiki.org/wiki/Extension:DynamicPageList_(Wikimedia) has an option `mode=gallery` which allows showing a gallery of images that match the query. Currently this only really works with files in the file namespace. Your task is to have it use the page image (from the PageImages extension https://www.mediawiki.org/wiki/Extension:PageImages ) for pages that are not file pages. It is recommended you have already done at least two tasks involving PHP coding of MediaWiki extensions that involved submitting code to gerritCurrently it just displays an empty space for pages not in the file namespace. The DynamicPageList_(Wikimedia) extension (also known as the "intersection" extension) https://www.mediawiki.org/wiki/Extension:DynamicPageList_(Wikimedia) has an option `mode=gallery` which allows showing a gallery of images that match the query.Prerequisites: * You have done several MediaWiki PHP coding related tasks, and are looking for a more complex PHP coding task * You have already setup MediaWiki locally * You have a basic knowledge of SQL and know what a LEFT JOIN is (e.g. You understand what the query `SELECT page_namespace, page_title, Currently this only really works with files in the file namespacep1.pp_value AS 'image' FROM page left join page_props p1 on p1.pp_page = page_id and p1.pp_propname = 'page_image_free' where page_id = 1931416;` means. Your task is to have it use the page image (from the PageImages extension https://www.mediawiki.org/wiki/Extension:PageImages ) for pages that are not file pages.If not, there are several sql tutorials online) To do thisIt is recommended you skim through https://www.mediawiki.org/wiki/Manual:Page_props_table and https://www.mediawiki.org/wiki/Manual:Database_access#Database_Abstraction_Layer to give you background as to how MediaWiki makes SQL queries, if you are not already familiar with this. However that page goes in a lot of detail, you need to modify two parts:so don't worry if you don't understand all of it. Where the SQL is generated (around line 441-ish of intersection/includes/DynamicPageListHooks.php ), provided you are in gallery mode, and the PageImages extension is loaded, you would have to add page_props to the $tables. You should use an alias so the key to the array is the alias name, and the value is `page_props`. You then need to add a join to $join for page_props. it should be a LEFT JOIN and the ON condition should be pp_propname => PageImages::PROP_NAME. You should then do the same thing with a different alias but for PageImages::PROP_NAME_FREE.To do this, You then haveyou need to add aliasforpageprops.pp_value (Probably also giving this an alias for clarify) to $fieldsmodify two parts of the dynamic page list extension: Where the SQL is generated (around line 441-ish of intersection/includes/DynamicPageListHooks.php ), provided you are in gallery mode, and the PageImages extension is loaded, you would have to add page_props to the $tables. You should use an alias so the key to the array is the alias name, and the value is `page_props`. You then need to add a join to $join for page_props. it should be a LEFT JOIN and the two ON conditions should be pp_propname = PageImages::PROP_NAME and page_id = pp_page (adding appropriate table aliases). You should then do the same thing with a different alias but for PageImages::PROP_NAME_FREE. You then have to add <alias>.pp_value to $fields (Probably also giving this an alias for clarify). Doing these modifications show change the query so that the database returns a column containing the "free" page image and a column containing the "non-free" page image Once the query is modified, we now have to make use of the modified query. Now you must adjust the gallery building code to use the alternative images for pages that are not in the file namespace. You then have to find the code that calls $gallery->add. Instead of the current code, if $title->getNamespace() !== NS_FILE you have to get the field from the page_prop (trying first the free version, and then not free version if free is null), create a title from it (use Title::makeTitleSafe), and add that instead of the current title. However if you are not adding the current title, you also have to make sure the link goes to the right place, so you have to provide the fourth argument to $gallery->add as a link to the original title, e.g. $title->getLinkURL() Once all this is done, test the code on your local wiki to make sure it all works. Make a commit with an appropriate commit message, and upload to gerrit.