This module provides Smarter Blox, which can be placed on pages via an include/exclude pattern based on the page's Drupal-relative URL matching a URL pattern in the include list and not matching a URL pattern in the exclude list.

Project Page:
https://drupal.org/sandbox/joe_bfstplk/2022611

GIT Repository:
git clone --branch master joe_bfstplk@git.drupal.org:sandbox/joe_bfstplk/2022611.git smarter_blox

Installation and Usage
Download and unzip in the modules folder, enable on the modules page. Comes set for four (4) Smarter Blox blocks by default. Admin page allows setting up more or less Smarter Blox blocks per administrator preference. See README.txt for details on usage.

Comments

TimTheEnchanter’s picture

TimTheEnchanter commented
Category: feature » support

Hello,
To start with you will need to move from master to a major version, here is a link to help. https://drupal.org/empty-git-master

Also you will need to address the issues seen here before you can get this approved. http://ventral.org/pareview/httpgitdrupalorgsandboxjoebfstplk2022611git

Take a look at the coding standards. Whitespace, line length, commenting, etc. have strict requirements. https://drupal.org/coding-standards

TimTheEnchanter’s picture

TimTheEnchanter commented
Status: Active » Needs work

Sorry I meant to mark needs work

TimTheEnchanter’s picture

TimTheEnchanter commented

It also looks like you have an empty directory named smarter_blox in the module folder.

The git command you have in this issue is incorrect, that command requires your password. The correct git command is
git clone --branch master http://git.drupal.org/sandbox/joe_bfstplk/2022611.git smarter_blox

Notice no user name in the command. When you update this to include a major version please change the git command to reflect the correct branch.

bogdanru’s picture

bogdanru commented

Hi,

In function smarter_blox_uninstall(), you should loop through your variables and delete them.
You could use something like this:

function smarter_blox_uninstall() {
  // Simple DB query to get the names of our variables.
  $results = db_select('variable', 'v')
    ->fields('v', array('name'))
    ->condition('name', 'smarter_blox_%', 'LIKE')
    ->execute();
  // Loop through and delete each of our variables.
  foreach ($results as $result) {
    variable_del($result->name);
  }
}

Cheers.

joe_bfstplk’s picture

joe_bfstplk
Primary language English
commented

Thanks for your helpful tips. Below you will find the most recent version, hopefully much more compliant with Drupal Coding Standards.

I have a little concern for the overhead in loading all of the content nodes to look at field values in 'field_show_in_page', 'field_hide_from_page', and 'field_show_in_blox' so that only the necessary nodes can be loaded? Is this even the right place to ask that question? Sorry for all the newbie questions.

Thanks!

--Dale

----

Issue Summary

This module provides Smarter Blox, which can be placed on pages via an include/exclude pattern based on the page's Drupal-relative URL matching a URL pattern in the include list and not matching a URL pattern in the exclude list.

Project Page:
https://drupal.org/sandbox/joe_bfstplk/2022611

GIT Repository:
git clone --branch smarter_blox-7.x-1.x joe_bfstplk@git.drupal.org:sandbox/joe_bfstplk/2022611.git smarter_blox

Installation and Usage
Download and unzip in the modules folder, enable on the modules page. Comes set for four (4) Smarter Blox blocks by default. Admin page allows setting up more or less Smarter Blox blocks per administrator preference. See README.txt for details on usage.

PA robot’s picture

PA robot commented
Status: Needs work » Closed (won't fix)

Closing 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.