Integrating Workbench Access with IMCE

Ah, solved it. I was variable_set'ing the wrong variable in imce_admin_submit.

Still would like feedback on my code.

Also, isn't there a way I could do this all in a custom module? So that I wouldn't have to modify IMCE core code.

Forms API should work for the forms, I had thought about it, but not tried.

What about functions like imce_user_profile()? I had to modify that function.

Specifically, I had to add:

if(imce_workbench_access_exists ())
  { 
    $imce_workbench_sections_profiles = variable_get('imce_workbench_sections_profiles', array());
    $workbench_access_user_sections = workbench_access_get_user_tree();
    foreach ($imce_workbench_sections_profiles as $sid => $section) {
        if(isset($workbench_access_user_sections[$sid]))
        {
            $conf = $section;
            break;
        }
    }
  }

after:

//handle regular users
  $roles_profiles = variable_get('imce_roles_profiles', array());
  $conf = array();
  foreach ($roles_profiles as $rid => $role) {
    if (isset($user->roles[$rid])) {
      $conf = $role;
      break;
    }
  }

in imce_user_profile().

Here are the diff files for imce.module and imce.admin.inc. Also, a new file called imce.workbench.inc.

What works on my dev instance:

• Assigning profiles to Workbench Access taxonomy sections.
• IMCE File Browser only showing the directory assigned to the profile for the section a user is an editor in.

What I am working on, and would love help with:

• Pulling what I've done in IMCE core out and creating a custom module.
• Showing users multiple directories when they are assigned to mulitple sections in Workbench Access (currently, it only shows the first one that has a profile assigned to it.)
• Dealing with sections created from menus.

ok, I have a custom admin page set up and working. Thanks for that tip.

Now, I'm trying to figure out how to get my access control logic into imce_user_profile(). Maybe I've just been staring at the code for too long, but I don't see how adding a custom menu item would help with that. Unless I wanted to totally override all the code in the function. I just want it to check variable_get('imce_workbench_sections_profiles') for access rights.

Thanks again!