php - Cakephp not redirecting after sending email through CakeEmail -

-

in cakephp 2.x, 1 of controller actions, trying send email user's every time new record created.

updated drmonkeyninja's suggestions:

// newsletterscontoller.php  public function add() {   if($this->request->is('post')) {     $this->newsletter->create();    if($this->newsletter->save($this->request->data)) {       $this->session->setflash(__('your newsletter has been submited.'));       $this->redirect(array('action' => 'view', $this->newsletter->id));   }   else {       $this->session->setflash(__('unable add post.'));       return $this->redirect(array('action' => 'add'));   }  } }  // newsletter.php public function aftersave($created, $options = []) {   parent::aftersave($created, $options);   if ($created === true) {     $newsletter = $this->findbyid($this->id);      // users email address.     $emails = classregistry::init('user')->find(         'list',         array(             'fields' => array(                 'user.id',                 'user.email'             ),             'conditions' => array(                 'user.email <>' => null             )         )     );      $email = new cakeemail('gmail');     $email->from(array('me@example.com' => 'caprock portal'));     $email->to($emails);     $email->subject($newsletter['newsletter']['title']);     $email->message($newsletter['newsletter']['content']);     try {        $email->send();     } catch (exception $exception) {         $this->log($exception);     }   } }

as can see code snippet, use cakeemail send every user email containing newsletter created in action. unfortunately reason, every time cakeemail finishes sending email, cakephp ignores redirect request , proceeds render blank view (route still add). have verified cakeemail function commenting , verifying redirection starts work again. nothing caught in try-catch block either.

one of assumptions cause of problem email headers interfering redirect headers action. inspected network requests through browser, nothing seems sent out, except original post request add function.

it better send email newsletter model in aftersave() callback:-

// app/model/newsletter.php  app::uses('cakeemail', 'network/email');  public function aftersave($created, $options = []) {     parent::aftersave($created, $options);     if ($created === true) {         $newsletter = $this->findbyid($this->id);          // users email address.         $emails = classregistry::init('user')->find(             'list',             array(                 'fields' => array(                     'user.id',                     'user.email'                 ),                 'conditions' => array(                     'user.email <>' => null                 )             )         );          $email = new cakeemail('gmail');         $email->from(array('me@example.com' => 'caprock portal'));         $email->to($emails);         $email->subject($newsletter['newsletter']['title']);         $email->message($newsletter['newsletter']['content']);         try {            $email->send();         } catch (exception $exception) {             $this->log($exception);         }     } }

your controller action be:-

// app/controller/newsletterscontroller.php public function add() {     if ($this->request->is('post')) {         $this->newsletter->create();         if ($this->newsletter->save($this->request->data)) {             return $this->redirect(array('action' => 'view', $this->newsletter->id));         } else {             $this->session->setflash(__('unable add post.'));             return $this->redirect(array('action' => 'add'));         }     } }

if view still not rendering temporarily disable aftersave() callback check controller part of code working expected when saving newsletter.

note can filter out users without email address when retrieve users database. using find('list') don't need mess foreach loop.

you don't need use $this->newsletter->getlastinsertid(); $this->newsletter->id should have been set @ time of save.

cake's debug() method better use when debugging variables var_dump() incidentally won't work in example redirecting after has been output!


Comments

Post a Comment

Popular posts from this blog

python - TypeError: start must be a integer -

-
i trying draw 5 turtles, getting typeerror on line 25. here code: import turtle wn = turtle.screen() redrose = turtle.turtle() color = input("what background color be?") fillcolor_f = input("what color of rose be?") redrose.hideturtle() redrose.speed(30) redrose.penup() redrose.left(180) redrose.forward(175) redrose.right(90) redrose.forward(30) redrose.right(90) redrose.pendown() def drawrose(red): redrose.color("pink") redrose.fillcolor(fillcolor_f) redrose.fill(true) in range(red): redrose.forward(i) redrose.right(49) in range(5): drawrose(redrose) redrose.penup() redrose.forward(350) redrose.right(144) redrose.pendown() redrose.fill(false) drawrose(50) wn.bgcolor(color) i trying draw 5 roses, produces errors. doing in interactivepython.org. you recursively calling drawrose wrong parameter. on line 23 ( for in range(red): ) expect red integer...
Read more

c# - DevExpress RepositoryItemComboBox BackColor property ignored -

-
i have following code almost works properly; private void cbstatus_drawitem(object sender, listboxdrawitemeventargs e) { string item = e.item string; if (item != null) { switch (item) { case "1": e.appearance.forecolor = color.green; e.appearance.backcolor = color.green; break; case "2": e.appearance.forecolor = color.orange; e.appearance.backcolor = color.orange; break; case "3": e.appearance.forecolor = color.red; e.appearance.backcolor = color.red; break; } } } when dropdown shown, items forecolor correct, backcolor remains whatever backcolor of theme is; i.e. if i've got set dark theme, backcolor dark, cells in gridview, rather being green/orange/red. i've tried setting e.appearance.options.usebackcolor trying set e....
Read more

django - Creating multiple model instances in DRF3 -

-
1) create multiple model instances 1 api call, asked here: how create multiple model instances django rest framework? tried solution named in link, no success. i trying upload multiple files in 1 api call. result: files uploaded (only once overwrote perform_create) 1 instance created (if send 2 files, latter created instance). my code: class fileuploadserializer(serializers.modelserializer): def __init__(self, *args, **kwargs): many = kwargs.pop('many', true) super(fileuploadserializer, self).__init__(many=many, *args, **kwargs) class meta: model = fileupload read_only_fields = ('created', 'datafile', 'owner') class fileuploadviewset(viewsets.modelviewset): queryset = fileupload.objects.all() serializer_class = fileuploadserializer parser_classes = (multipartparser, formparser, ) def perform_create(self, serializer): file_list = self.request.data.getlist('file') ...
Read more