Suggestion for email library

[bfarber]

    {        if ( $this->template == "" )        {            $this->error++;            $this->fatal_error("Could not build the email message, no template assigned", "Make sure a template is assigned first.");        }                $this->message = $this->template;                //-----------------------------------------        // Add some default words        //-----------------------------------------                $words['BOARD_ADDRESS'] = $this->ipsclass->vars['board_url'] . '/index.' . $this->ipsclass->vars['php_ext'];        $words['WEB_ADDRESS']   = $this->ipsclass->vars['home_url'];        $words['BOARD_NAME']    = $this->ipsclass->vars['board_name'];        $words['SIGNATURE']     = isset($this->ipsclass->vars['signature']) ? $this->ipsclass->vars['signature'] : '';                //-----------------------------------------        // Swap the words        //-----------------------------------------                $this->_words = $words;                $this->message = preg_replace_callback( "//", array( &$this, '_swap_words' ), $this->message );        /* AKAC CHANGE -  */        $this->lang_subject = preg_replace_callback( "//", array( &$this, '_swap_words' ), $this->lang_subject );        /* AKAC CHANGE -  */                $this->_words = array();                $this->message = $this->clean_message( $this->message );    }

    function build_message( $words )

Yes, we make language changes so that the subjects make sense. A subject of "Topic Reply Notification" when you have 50 of them is hard to manage. A subject of "Topic Reply Notification [ Sleep After Idling No Longer Works] Posted by: bloosqr ( site )" is far more usable.

Since you allow the body templates to have this customization it makes sense for the subject to be as well. Especially since the template page makes no mention that only certain fields are customizable.

Its a simple modification that makes the board much more usable and adds no risk.