The Tile module allows site administrators and themers to specify icons for use as tile on startscreen of windows 8.1.

Extra theme settings are provided so the tile can be configured for each theme, similar to the logo and shortcust icon settings provided by Drupal core.

Project sandbox

Git repository

git clone tile

Reviews of other projects

#12 coder-results.txt2.01 KBklausi
tile.gif13.06 KBdrupalfan79


drupalfan79’s picture

Status: Active » Needs review
PA robot’s picture

Status: Needs review » Needs work

There are some errors reported by automated review tools, did you already check them? See

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

drupalfan79’s picture

Issue summary: View changes

add reviews of other projects

drupalfan79’s picture

Status: Needs work » Needs review

Fixed errors reported by automated review tools from

Martin Schauer’s picture

Hi drupalfan79!

I had a look at your project.

- At first the git-repository in the issue-description is your private one, use "git clone tile" instead.

- The path to the theme-settings in D7 should be "admin/appearance/settings", not "admin/build/themes/settings" as explained in your README.txt.

- Feel free to also add the line "configure = admin/appearance/settings" to the .info-file, to add a configure-shortcut next to your module in Drupal's module-list.

- On the theme-overview-page tile_get_theme() returns "settings", because this form is not for a specific theme, which leads to 3 notices:

Notice: Undefined index: setting in theme_get_setting() (line 1431 of includes/
Notice: Trying to get property of non-object in theme_get_setting() (line 1468 of includes/
Notice: Trying to get property of non-object in theme_get_setting() (line 1478 of includes/

On saving the form the notices get duplicated, but data gets saved, so you only should get rid of these notices.

- Another error lies somewhere within the paths of uploaded images. I didn't look through why this error occurs, but after uploading an image and saving the theme settings, the meta-tags are set correctly. If I then save the form again, the 4 meta-tags "msapplication-square70x70logo", "msapplication-wide310x150logo" etc. disappear.

- Overall some additional inline-comments could enhance the readability of your code.

Anyway you did a good job creating this module, maybe you could think about working together with @konstantin.komelin on pinned_site ( somewhere in the future, that would be a cool merge.


Martin Schauer’s picture

Status: Needs review » Needs work
Martin Schauer’s picture

Issue summary: View changes

update git repository

drupalfan79’s picture

Status: Needs work » Needs review

@Martin, thanks for your great, detail review. it is fixed.

AmazingDreams’s picture

Hi, I took a look at your code.

At first I ran Coder, it came up with one error:

comment_docblock_fileFile: @file block missing (Drupal Docs) [comment_docblock_file]

Then I quickly scanned your .module file, there are zero inline comments! I'm sure they are needed somewhere.

Then I read your code, and have some questions for you which you should answer in your code.

  1. What is happening in <?php function tile_get_theme()?
  2. Line 50/51 you are doing something with a tile path... why? What are you doing there? I have to carefully read it to figure it out,
    As with #1 you use rather vague variables like $i and $j, why not use $first_occurance, $last_occurance?
  3. What happens when the file is not a gif/png/jpeg?
  4. The file stuff is a bit hard to read due to missing comments. The way I read it: if ( file is uploaded and valid ): move the file and do create images; otherwise: Do nothing with the file but create new images anyway Please add comments as to why this happens

Some things became clear when I saw the frontend, but what if the frontend is not working for someone and someone has to work her way through ondocumented code to figure out where the error is coming from.


  • You have a file form which only accepts gif/jpeg/png, why not 'hard-code' it into the HTML? You could use #attributes This will make it easier for your users to find the file.
  • You handle form submission in the form validation, yet you seem to do no validation (apart from the file stuff)

And I'm sure it works, I can't test it though as I use linux.

abghosh82’s picture

Hi drupalfan79,

I did a manual review of your module all looks fine, accept a few minor cosmetic suggestions.

The function 'tile_get_theme()' can be documented bit more for the logic used.
Please consider using short but little descriptive names of variables instead of $i and $j.

For the function 'tile_create_images', first the block comment of the function has typo 'lite', instead of 'tile' I believe. Any reason why it is restricted to 4 different size images, if the user wants more?
Here also please consider changing $i to a meaningful name. This actually helps to understand the code even if comments are not there. :)

Rest all looks fine.

drupalfan79’s picture

Hi AmazingDreams,

  1. @file block is already in the js file, but the coder module doesn't detect it, it's the coder module bug:
  2. inline comments are added.
  3. tile_get_theme function is used to get theme name, such as garland, seven.
  4. not gif/jpg/png files will be ingored.
  5. we can't restrict file extendsion in html.
drupalfan79’s picture

Hi abghosh82,

  1. inline comments are added in tile_get_theme function
  2. lite is changed to tile
  3. windows8.1 tile asks four sizes images.
drupalfan79’s picture

Issue summary: View changes

update git repository

drupalfan79’s picture

Issue summary: View changes

add another project review

drupalfan79’s picture

Issue tags: +PAReview: review bonus

Add tag PAReview: review bonus

klausi’s picture

Assigned: Unassigned » chx
Status: Needs review » Reviewed & tested by the community
Issue tags: -PAReview: review bonus
2.01 KB

Thank you for reviews. When finishing your review comment also set the issue status either to "needs work" (you found some problems with the project) or "reviewed & tested by the community" (you found no major flaws).

Review of the 7.x-1.x branch:

This automated report was generated with, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.

manual review:

  1. Commit messages are broken, you are always using the text "Initial" . Please provide meaningful messages, see
  2. The Git commits are not connected to your user account. You need to specify an email address. See and
  3. start screen of windows 8.1? How does that work, what does the module do? Please improve your project page, see

Otherwise looks RTBC to me. Removing review bonus tag, you can add it again if you have done another 3 reviews of other projects.

Assigning to chx as he might have time to take a final look at this.

drupalfan79’s picture

Hi klausi,
Thanks for your great review.

  1. issues are fixed.
  2. Latest commit messages is improved with meaningful messages.
  3. Latest commit is connected to my user account.
  4. Added 'What is tile?' in the project page.
drupalfan79’s picture

Issue summary: View changes

add reveiw of other projects

chx’s picture

Assigned: chx » Unassigned
Status: Reviewed & tested by the community » Fixed

I did review the code; it looks okay; although there's a bug: the file is moved but the $file object is not updated with the new uri and is not resaved and file usage is not added. Other than this bug, it's OK, so

Thanks for your contribution, drupalfan79 !

I updated your account so you can promote this to a full project and also create new projects as either a sandbox or a "full" project.

Here are some recommended readings to help with excellent maintainership:

You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and stay involved!

Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

Thanks to the dedicated reviewer(s) as well.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

add another project review