Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
theme
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
24 May 2013 at 22:25 UTC
Updated:
27 Nov 2013 at 22:26 UTC
Jump to comment: Most recent

Comments
Comment #1
PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://ventral.org/pareview/httpgitdrupalorgsandboxvelaio2000772git
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.
Comment #2
gcardona commentedHello Drupal team testers, we fixed all the standard code issues. Please check again and send us your comments and questions....
Regards,
Gustavo Cardona Ramirez and VELAIO team... :-)
Comment #3
pplantinga commentedIt looks like you're trying to make this theme commercial
There's a lot of work required to make this theme ready for prime time.
Notice: Undefined variable: title in include() (line 5 of /var/www/sites/all/themes/business_reinvent/templates/region--content.tpl.php).
Notice: Undefined variable: tabs in include() (line 11 of /var/www/sites/all/themes/business_reinvent/templates/region--content.tpl.php).
Notice: Undefined variable: action_links in include() (line 12 of /var/www/sites/all/themes/business_reinvent/templates/region--content.tpl.php).
Notice: Undefined variable: feed_icons in include() (line 14 of /var/www/sites/all/themes/business_reinvent/templates/region--content.tpl.php).
Notice: Undefined variable: breadcrumb in include() (line 3 of /var/www/sites/all/themes/business_reinvent/templates/zone--content.tpl.php).
Notice: Undefined variable: messages in include() (line 6 of /var/www/sites/all/themes/business_reinvent/templates/zone--content.tpl.php).
Notice: Undefined variable: doctype in include() (line 1 of /var/www/sites/all/themes/business_reinvent/templates/html.tpl.php).
Notice: Undefined variable: rdf in include() (line 4 of /var/www/sites/all/themes/business_reinvent/templates/html.tpl.php).
Notice: Trying to get property of non-object in include() (line 4 of /var/www/sites/all/themes/business_reinvent/templates/html.tpl.php).
Notice: Undefined variable: rdf in include() (line 4 of /var/www/sites/all/themes/business_reinvent/templates/html.tpl.php).
Notice: Trying to get property of non-object in include() (line 4 of /var/www/sites/all/themes/business_reinvent/templates/html.tpl.php).
Notice: Undefined variable: rdf in include() (line 5 of /var/www/sites/all/themes/business_reinvent/templates/html.tpl.php).
Notice: Trying to get property of non-object in include() (line 5 of /var/www/sites/all/themes/business_reinvent/templates/html.tpl.php).
As for the complex instructions, having the user execute raw sql is probably a bad idea. It will overwrite any data that they may have already, and it looks like it changes drupal's database structure, which could mess with modules and many other things.
Even if you end up fixing all these things, I'm not confident that this project will be unique enough to differentiate it from what Omega/Alpha already do, but feel free to keep trying. If you feel there is something unique about the project, let us know what it is.
Comment #4
PA robot commentedClosing due to lack of activity. Feel free to reopen if you are still working on this application.
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #5
gcardona commentedHello Drupal team, we fixed and modified again the comments and suggestions sent for the testers.
Please check the theme and give us your feedback.
Regards and thanks,
--
Gustavo Cardona Ramirez
Comment #6
kscheirerWith your .sql file, it looks like you're trying to combine a theme + content and configuration, which is not going to work. Instead, Drupal does have the concept of Features and Install Profiles that should fit your needs better.
If you agree, I think it's easiest to close this application and start a fresh one.
----
Top Shelf Modules - Crafted, Curated, Contributed.
Comment #7
gcardona commentedHello kscheirer, please check the video installation. We fixed all problems after applied the SQL. Our propose is a site ready to use with a good Look and feel. This "basic installation" is for all Drupal installations but the "advanced installation" is only for new drupal installations.
For many, Drupal new users is difficult configure a theme with demo data, set parameters, install and configure each module and so on. We are trying to provide an alternative to get a good design out the box.
Please check again our theme try to install.
Regards,
Gustavo
Comment #8
PA robot commentedClosing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.