diff --git a/app/controllers/webmail_controller.rb b/app/controllers/webmail_controller.rb index f11d02d..22402db 100644 --- a/app/controllers/webmail_controller.rb +++ b/app/controllers/webmail_controller.rb @@ -150,7 +150,7 @@ class WebmailController < ApplicationController # delete temporary files (attachments) @mail.delete_attachments() - return render("webmail/webmail/mailsent") + render :action => "webmail/mailsent" elsif operation == _('Add') @mail = create_mail attachment = CDF::Attachment.new(@mail) diff --git a/app/helpers/webmail_helper.rb b/app/helpers/webmail_helper.rb index 3602989..2f287d4 100644 --- a/app/helpers/webmail_helper.rb +++ b/app/helpers/webmail_helper.rb @@ -4,59 +4,59 @@ require 'mail2screen' module WebmailHelper include Mail2Screen def link_folders - link_to(_('Folders'), :controller=>"/webmail/webmail", :action=>"messages") + link_to(_('Folders'), :controller=>"webmail", :action=>"messages") end def link_send_mail - link_to(_('Compose'), :controller=>"/webmail/webmail", :action=>"compose") + link_to(_('Compose'), :controller=>"webmail", :action=>"compose") end def link_compose_new - link_to(_('Compose new mail'), :controller=>"/webmail/webmail", :action=>"compose") + link_to(_('Compose new mail'), :controller=>"webmail", :action=>"compose") end def link_refresh - link_to(_('Refresh'), :controller=>"/webmail/webmail", :action=>"refresh") + link_to(_('Refresh'), :controller=>"webmail", :action=>"refresh") end def link_message_list - link_to(_('Message list'), :controller=>"/webmail/webmail", :action=>"messages") + link_to(_('Message list'), :controller=>"webmail", :action=>"messages") end def link_reply_to_sender(msg_id) - link_to(_('Reply'), :controller=>"/webmail/webmail", :action=>"reply", :params=>{"msg_id"=>msg_id}) + link_to(_('Reply'), :controller=>"webmail", :action=>"reply", :params=>{"msg_id"=>msg_id}) end def link_forward_message(msg_id) - link_to(_('Forward'), :controller=>"/webmail/webmail", :action=>"forward", :params=>{"msg_id"=>msg_id}) + link_to(_('Forward'), :controller=>"webmail", :action=>"forward", :params=>{"msg_id"=>msg_id}) end def link_flag_for_deletion(msg_id) - link_to(_('Delete'), :controller=>"/webmail/webmail", :action=>"delete", :params=>{"msg_id"=>msg_id}) + link_to(_('Delete'), :controller=>"webmail", :action=>"delete", :params=>{"msg_id"=>msg_id}) end def link_view_source(msg_id) - link_to(_('View source'), {:controller=>"/webmail/webmail", :action=>"view_source", :params=>{"msg_id"=>msg_id}}, {'target'=>"_blank"}) + link_to(_('View source'), {:controller=>"webmail", :action=>"view_source", :params=>{"msg_id"=>msg_id}}, {'target'=>"_blank"}) end def link_manage_folders - link_to(_('add/edit'), :controller=>"/webmail/webmail", :action=>"manage_folders") + link_to(_('add/edit'), :controller=>"webmail", :action=>"manage_folders") end def link_back_to_messages - link_to("«" << _('Back to messages'), :controller=>"/webmail/webmail", :action=>"messages") + link_to("«" << _('Back to messages'), :controller=>"webmail", :action=>"messages") end def link_mail_prefs - link_to(_('Preferences'), :controller=>"/webmail/webmail", :action=>"prefs") + link_to(_('Preferences'), :controller=>"webmail", :action=>"prefs") end def link_mail_filters - link_to(_('Filters'), :controller=>"/webmail/webmail", :action=>"filters") + link_to(_('Filters'), :controller=>"webmail", :action=>"filters") end def link_filter_add - link_to(_('Add filter'), :controller=>'/webmail/webmail', :action=>'filter_add') + link_to(_('Add filter'), :controller=>'webmail', :action=>'filter_add') end def folder_link(folder) @@ -71,12 +71,12 @@ module WebmailHelper end if folder.name == CDF::CONFIG[:mail_trash] (unseen_messages > 0 ? "" : "" ) << - link_to( fn, :controller=>"/webmail/webmail", :action=>"messages", :params=>{"folder_name"=>folder.name}) << - " " << link_to(_('(Empty)'), {:controller=>"/webmail/webmail", :action=>"empty", :params=>{"folder_name"=>folder.name}}, :confirm => _('Do you really want to empty trash?')) << + link_to( fn, :controller=>"webmail", :action=>"messages", :params=>{"folder_name"=>folder.name}) << + " " << link_to(_('(Empty)'), {:controller=>"webmail", :action=>"empty", :params=>{"folder_name"=>folder.name}}, :confirm => _('Do you really want to empty trash?')) << (unseen_messages > 0 ? "" : "" ) else (unseen_messages > 0 ? "" : "" ) << - link_to( fn, :controller=>"/webmail/webmail", :action=>"messages", :params=>{"folder_name"=>folder.name}) << + link_to( fn, :controller=>"webmail", :action=>"messages", :params=>{"folder_name"=>folder.name}) << (unseen_messages > 0 ? "" : "" ) end end @@ -94,7 +94,7 @@ module WebmailHelper short_fn(folder) else return short_fn(folder) + - (" " + link_to(_('(Delete)'), :controller=>"/webmail/webmail", :action=>"manage_folders", :params=>{"op"=>_('(Delete)'), "folder_name"=>folder.name})) + (" " + link_to(_('(Delete)'), :controller=>"webmail", :action=>"manage_folders", :params=>{"op"=>_('(Delete)'), "folder_name"=>folder.name})) end end @@ -135,19 +135,19 @@ module WebmailHelper end def link_filter_up(filter_id) - link_to(_('Up'), :controller=>"/webmail/webmail", :action=>"filter_up", :id=>filter_id) + link_to(_('Up'), :controller=>"webmail", :action=>"filter_up", :id=>filter_id) end def link_filter_down(filter_id) - link_to(_('Down'), :controller=>"/webmail/webmail", :action=>"filter_down", :id=>filter_id) + link_to(_('Down'), :controller=>"webmail", :action=>"filter_down", :id=>filter_id) end def link_filter_edit(filter_id) - link_to(_('Edit'), :controller=>"/webmail/webmail", :action=>"filter", :id=>filter_id) + link_to(_('Edit'), :controller=>"webmail", :action=>"filter", :id=>filter_id) end def link_filter_delete(filter_id) - link_to(_('Delete'), :controller=>"/webmail/webmail", :action=>"filter_delete", :id=>filter_id) + link_to(_('Delete'), :controller=>"webmail", :action=>"filter_delete", :id=>filter_id) end def page_navigation_webmail(pages) @@ -161,16 +161,16 @@ module WebmailHelper if pages.current == page nav << page.number.to_s << " " else - nav << link_to(page.number, :controller=>"/webmail/webmail", :action=>'messages', :page=>page.number) << " " + nav << link_to(page.number, :controller=>"webmail", :action=>'messages', :page=>page.number) << " " end end nav << "..." unless window_pages[-1].last? nav << "   " - nav << link_to(_('First'), :controller=>"/webmail/webmail", :action=>'messages', :page=>@pages.first.number) << " | " unless @pages.current.first? - nav << link_to(_('Prev'), :controller=>"/webmail/webmail", :action=>'messages', :page=>@pages.current.previous.number) << " | " if @pages.current.previous - nav << link_to(_('Next'), :controller=>"/webmail/webmail", :action=>'messages', :page=>@pages.current.next.number) << " | " if @pages.current.next - nav << link_to(_('Last'), :controller=>"/webmail/webmail", :action=>'messages', :page=>@pages.last.number) << " | " unless @pages.current.last? + nav << link_to(_('First'), :controller=>"webmail", :action=>'messages', :page=>@pages.first.number) << " | " unless @pages.current.first? + nav << link_to(_('Prev'), :controller=>"webmail", :action=>'messages', :page=>@pages.current.previous.number) << " | " if @pages.current.previous + nav << link_to(_('Next'), :controller=>"webmail", :action=>'messages', :page=>@pages.current.next.number) << " | " if @pages.current.next + nav << link_to(_('Last'), :controller=>"webmail", :action=>'messages', :page=>@pages.last.number) << " | " unless @pages.current.last? nav << "

" diff --git a/components/contacts/customer.rb b/app/models/customer.rb similarity index 88% rename from components/contacts/customer.rb rename to app/models/customer.rb index 27737c3..a9787a3 100644 --- a/components/contacts/customer.rb +++ b/app/models/customer.rb @@ -23,7 +23,7 @@ class Customer < ActiveRecord::Base end def check_for_domain(email) - if email and !email.nil? and !email.include?("@") + if email && !email.nil? && !email.include?("@") && CDF::CONFIG[:send_from_domain] email + "@" + CDF::CONFIG[:send_from_domain] else email diff --git a/app/views/webmail/compose.rhtml b/app/views/webmail/compose.rhtml index 8bee588..4f23c75 100644 --- a/app/views/webmail/compose.rhtml +++ b/app/views/webmail/compose.rhtml @@ -1,5 +1,5 @@

<%=_('Mailbox')%>

-
+