Download & Extend

Remove drupal_set_message() from API functions

Project:Units API
Version:6.x-1.0-alpha1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Why do you show messages in API functions that are supposed to be called from unknown places ? Would be nice to move them to standalone local function, that calls API functions. Also, optional (!!!) watchdog messages would be much better! So if I would want to convert units in my module, I didn't need to care about trash on the screen. My users don't want to know anything about APIs etc, and I think most users don't too.

Comments

#1

Status:active» fixed

All watchdog now: http://drupal.org/cvs?commit=319030

Thanks for catching this, crea!

#2

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.