Remove dsm() from code

This bug report has not been active for six months. In an effort to clean-up the issue queue this item has been closed. If your modules are current and the report is still relevant please feel free to change the Status back to active.

Only a few places needs review

ctools# grep -r "dsm" ./
./ctools.module:  dsm('should not be called!');
./ctools.module:  dsm('should not be called!');
./page_manager/page_manager.module:      dsm($retval);
./page_manager/page_manager.admin.inc:        dsm($path);
./page_manager/page_manager.admin.inc:        dsm($item);
./page_manager/page_manager.admin.inc:        dsm($attributes['class']);
./includes/stylizer.inc://  dsm($processor->message_log);

Why keep any of this? I'd say remove it completely.

Suppose this commented out code is a thing of a past.
Suppose we need to ping someone from ctools team to carefully review, probably this clean-up could be deeper.

can we get this commited?

It was a bad idea of me to use dsm() to point out these errors, but pounard is right -- it's better for developers to get explicit errors.

I don't remember why those functions are still there at this point. Possibly worth doing a little code archeaology to see.

so... let's change them to trigger_error()?

Throw named exceptions.

Okay, I went through the list. I removed the no longer needed context_cache functions. I replaced a dsm about $attributes['class'] with a correction to just make it an array. (Experience has shown that there's Always One More 'class' setting that was not updated to the array notation).

I left in the commented out dsm()s. In a way, they're a reminder of something I had intended but haven't gotten back to. Maybe someday I or someone will, and the commented code at least says something about it. Removing it guarantees nothing will ever happen there.

Committed and pushed.