WordPress.org

Make WordPress Themes

#31709 closed theme (not-approved)

THEME: OpenTute+ – 1.0.1

Reported by: kishores Owned by: cristiano.zanca
Priority: new theme Keywords: theme-opentute
Cc: babunsahoo@…

Description

Change History (26)

#1 @basantakumar
21 months ago

  • Owner set to basantakumar
  • Status changed from new to reviewing

#2 @grapplerulrich
21 months ago

  • Owner basantakumar deleted

I am sorry this review is taking so long. Sometimes people are unable to carry on the review, this may have happened this time. As a result, I am going to add this to the new queue again as a priority. This may mean it still takes time, but will prevent this ticket being held for so long by a reviewer that isn't able to carry on. Thanks for your patience.

If you are the reviewer and able to do this review, please carry on and request you get added back in Slack #themereview.

#3 @basantakumar
20 months ago

Here I found some error in the theme.

  • Footer copyright is hardcoded. Please fix it.
  • No Text domain defined in stylesheet
  • Theme uses option tree plugins for theme functionality. It should handle through theme customizations.
  • Theme uses twitter bootstrap but no credit given
  • The spelling of WordPress is missed few places ex- WordPres
  • Theme having a plugin included (option tree)

Theme check report
==============================
Now your theme has passed the basic tests you need to check it properly using the test data before you upload to the WordPress Themes Directory.

Make sure to review the guidelines at Theme Review before uploading a Theme.

Codex Links

Theme Development
Themes and Templates forum
Theme Unit Tests
INFO: More than one text-domain is being used in this theme. This means the theme will not be compatible with WordPress.org language packs.
The domains found are opentute, option-tree
INFO: option-tree/includes/ot-functions-settings-page.php The theme appears to use include or require. If these are being used to include separate sections of a template from independent files, then get_template_part() should be used instead.
INFO: option-tree/includes/ot-functions-docs-page.php The theme appears to use include or require. If these are being used to include separate sections of a template from independent files, then get_template_part() should be used instead.
Line 751: require( trailingslashit( get_template_directory() ) . \'includes/meta-boxe
Line 834: require( trailingslashit( get_template_directory() ) . \'option-tree/ot-loa
Line 851: require( trailingslashit( get_template_directory() ) . \'includes/theme-opt

This ticket was mentioned in Slack in #themereview by basantam. View the logs.


20 months ago

This ticket was mentioned in Slack in #themereview by todiadiyatmo. View the logs.


20 months ago

#6 @themetracbot
19 months ago

  • Summary changed from THEME: OpenTute+ – 1.0 to THEME: OpenTute+ – 1.0.1

#7 @kishores
18 months ago

@basantakumar Please someone review it :)

#8 @grapplerulrich
18 months ago

You can see your position in the queue here https://themes.trac.wordpress.org/report/2

You can help make it faster by doing reviews yourself.

#9 @rabmalin
16 months ago

Hello,

Thanks for uploading theme. When your theme reaches to the top of this Review Queue ( https://themes.trac.wordpress.org/report/2 ), reviewer will be assigned and review process will follow procedures found in the Theme Handbook - https://make.wordpress.org/themes/handbook/review/required/ . Ticket will be closed as not-approved if theme has more than 5 distinct issues. So, please check your theme against Theme Review Guidelines to avoid closing of ticket.

Note: This is not a review.

#10 @themetracbot
16 months ago

  • Owner set to bitcoinass

#11 @grapplerulrich
15 months ago

  • Owner bitcoinass deleted

I am sorry this review is taking so long. Sometimes people are unable to carry on the review, this may have happened this time. As a result, I am going to add this to the new queue again as a priority. This may mean it still takes time, but will prevent this ticket being held for so long by a reviewer that isn't able to carry on. Thanks for your patience.

If you are the reviewer and able to do this review, please carry on and request you get added back in Slack #themereview or you can take on another review when you have time again.

#12 @themetracbot
15 months ago

  • Owner set to piyoko

#13 @grapplerulrich
15 months ago

  • Owner piyoko deleted

I am sorry this review is taking so long. Sometimes people are unable to carry on the review, this may have happened this time. As a result, I am going to add this to the new queue again as a priority. This may mean it still takes time, but will prevent this ticket being held for so long by a reviewer that isn't able to carry on. Thanks for your patience.

If you are the reviewer and able to do this review, please carry on and request you get added back in Slack #themereview or you can take on another review when you have time again.

#14 @themetracbot
15 months ago

  • Owner set to cristiano.zanca

#15 @cristiano.zanca
15 months ago

Hi @kishores ,
I picked up the review of your theme.
The review process follows procedures found here https://make.wordpress.org/themes/handbook/review/.

If a requirement or anything else is not clear in the review, please comment in the ticket.

here you can see the theme review final queue

https://themes.trac.wordpress.org/query?status=approved&group=priority&col=id&col=summary&col=reporter&col=owner&col=priority&col=changetime&col=keywords&report=24&order=changetime

#16 @cristiano.zanca
15 months ago

Hi @kishores ,

these are the first issues I've found:

1)
wp_footer call should be right before the closing body tag. at file footer.php, line 93

2)
Script tags should be included on wp_enqueue_scripts or admin_enqueue_scripts instead of embedded directly at file header.php, line 23:

<script src="<?php echo esc_url( get_template_directory_uri() ); ?>/js/html5shiv/html5shiv.min.js?ver=3.7.3" type="text/javascript"></script>

3) this message appears with debug active:

NOTICE: wp-content/plugins/our-team-enhanced/inc/class/class.smartcat-team.php:661 - Undefined index: height
require('wp-blog-header.php'), require_once('wp-includes/template-loader.php'), include('/themes/opentute/page.php'), get_header, locate_template, load_template, require_once('/themes/opentute/header.php'), wp_head, do_action('wp_head'), call_user_func_array, SmartcatTeamPlugin->sc_custom_styles

Kind Regards

This ticket was mentioned in Slack in #themereview by cristiano.zanca. View the logs.


15 months ago

This ticket was mentioned in Slack in #themereview by cristiano.zanca. View the logs.


15 months ago

This ticket was mentioned in Slack in #themereview by cristiano.zanca. View the logs.


15 months ago

#20 @rabmalin
15 months ago

Hello @kishores Please respond within 7 days. Otherwise ticket will be closed as not approved. Today is day 6.

#21 @kishores
15 months ago

sorry for the delay. I will update soon.

#22 @cristiano.zanca
15 months ago

Hi @kishores ,

how things are going?

ten days has passed and nothing has been updated, have you some kind of trouble?

are you working on it?

Communication is important, please answer soon

Kind Regards

#23 @kishores
15 months ago

Hello @cristiano.zanca

I was away. So I could not fix. Please give me some more days. I will fix and upload again.

Thanks
Kishore

#24 @cristiano.zanca
14 months ago

Hi @kishores ,

2 weeks ago I've started review your theme.

you asked me to wait some days because you where away, and after that message 6 days passed.

What do you really want to do?

If nothing happens I'm sorry but I must close the review in a couple of days

Kind Regards

#25 @kishores
14 months ago

@cristiano.zanca sorry for delay. Perhaps I can not commit to this project due to some personal issues. You may close the ticket. Thanks for your time.

-Kishore

#26 @cristiano.zanca
14 months ago

  • Resolution set to not-approved
  • Status changed from reviewing to closed

@kishores ok,

I'm very sorry for waiting 3 weeks to read this.

Kind Regards

Note: See TracTickets for help on using tickets.