Ability to add only one occurrence of a block/widget to a homebox

I open to doing something like this as I don't see much use case for adding a single block multiple times. However, since this is used for the d.org dashboard, I want to be careful that we don't do something that might break it.

Come Monday, I'll check with drumm to see if / how this use case would break drupal.org.

Rhymes
How did you do this?
Lowell

Who's better than you!

subscribing.

The attached patch allows you to set the maximum number of times each block can be added. If 0 or undefined you can add an unlimited number. (current behavior)

The use case I have for adding more then one block is the block can be filtered in the settings so you can have multiple blocks with different filters.

I'm not fully happy with the term "max to show". If you come up with a better term use it. I'm not going to worry about something the site users won't see.

I added the Max to Show field to the block settings in the homebox layout form.

You can also define the maxToShow in the hook_homebox()
$pages['userdashboard'] = array(
...
'blocks' => array(
'myDashboard_block' => array(
'module' => 'myDashboard',
'delta' => 'block',
'region' => 1,
'movable' => 1,
'status' => 1,
'open' => 1,
'closable' => 1,
'title' => '',
'weight' => -7,
'maxToShow' => 3,
),
...

Here's something to consider. I'm using a 6.3 version and the blocks that are currently used have the class 'used'. You can set your CSS for display:none. As a quick way to test you could open your console of Chrome/Firebug and try this jquery: $(".used").hide() and the 'used' blocks will disappear. I do like the direction of the patch above though, because, as @rocnhorse noted there are use cases (only filters from what I can think of now) that will want more than one block to be there.

Patch from #7 updated for latest dev version, the hiding/showing is now done using to count/max count.

How about version 7 of Homeboxes. How can I prevent users of Homeboxes 7.x-2.0-beta6 to add duplicate homeboxes?

#9 works perfect!

I just add the standart occurence of 1 to the maxtoshow var. I think this is a good point to start for the most users.

On my research I overlooked this issue. Now I have a similar fix. The difference is to not a amount "max to show" but a simple checkbox to allow multiple.

Another problem was, that if a widget does not have content, it is not marked as used and so the add link won't be hidden and may be add multiple times ... but won't show because blocks with no content won't show in homebox.
I moved $allowed_blocks[$block->module][$block->delta]['used'] = TRUE; to another place outside of the condition if (!is_null($block)).

Last problem related to this issue I fixed: If a user clicked multiple times on the add block link, nothing prevented that the block will be add multiple times.

We're now using this (homebox-hide-addlinks-1066304-12.patch) and are having no problems with it. Nice work!

I am using version 6.x-3.0-beta5 and desire this functionality. Will the patch in #7 or #12 work for this version?

@bturoque: You will find out when you try the patch. Pleas leave a commend if it works or don't work.

The patch in #12 prevents new blocks that have been added with the "visible" option unchecked from being added to a user's dashboard.

This is because the code uses the 'open' property and not the 'status' property.

Here's the code in the patch (function homebox_add_block() approx line 1312:

 while (isset($user_blocks[$block_key])) {
    if (empty($user_blocks[$block_key]['open'])) {
      // A closed user block gots opened again. Add it with same color and to
      // the same region where it was before but on the bottom, because weight
      // likely changed within all blocks so we can't be sure where it
      // was before.

Correct code using 'status' instead of 'open' on the if(empty... line:

while (isset($user_blocks[$block_key])) {
    if (empty($user_blocks[$block_key]['status'])) {
      // A closed user block gots opened again. Add it with same color and to
      // the same region where it was before but on the bottom, because weight
      // likely changed within all blocks so we can't be sure where it
      // was before.

@osopolar Patches in #7 and #12 are for the 7.x version.

Thanks Osopolar,

Commend #12 patch is work like charm for me. :)

The patch allows for the removal of an already used block from the Add a block link however when I remove a block (by clicking on x) it does not come back up in the list of available blocks. User needs to reload the page in order to get access to that block.

#12 does not apply to the current version.

I created a new patch for the current version.
See included patch and interdiff.

New patch, without drush packaging information. Ignore #22
This is a reworked version of #12

From #2135291: [Policy, no patch] PHP 5.4 short array syntax coding standards for Drupal 8:

Please note, short array syntax is unsupported in versions of PHP prior to 5.4. This means that Drupal 7 core and Drupal 7 contributed projects without an explicit PHP 5.4+ requirement must use long array syntax.

+++ b/homebox.module
@@ -33,8 +33,8 @@ function homebox_menu() {
     foreach ($pages as $page) {
       $items[$page->settings['path']] = array(
-        'title' => $page->settings['title'],
         'title callback' => 'homebox_build_title',
+        'title arguments' => [$page->settings['title']],
         'page callback' => 'homebox_build',
         'page arguments' => array($page),
         'access callback' => '_homebox_user_access_view_homebox',
@@ -58,7 +58,7 @@ function homebox_menu() {

@@ -58,7 +58,7 @@ function homebox_menu() {
     if ($page) {
       $items['user/%user/' . $page->settings['path']] = array(
         'title callback' => 'homebox_build_title',
-        'title arguments' => array($page->settings['title']),
+        'title arguments' => [$page->settings['title']],
         'page callback' => 'homebox_build',

+++ b/homebox.admin.inc
@@ -64,7 +64,7 @@ function homebox_admin_page($form, &$form_state, $page = FALSE) {
+    '#description' => t('Can be positioned in <a href="!url">Menus admin</a>.', ['!url' => url('admin/structure/menu')]),

Thanks for the review, I guess a new patch is needed then.

Here's the new patch.

Looked at it again. This should be the right patch.
The interdiff should only be named 24-29.

We'll implement something like that in 3.0.x. Closing this for Drupal 7.