Witam, chciałbym aby email wysyłał wszystkie opcje, które spełniają warunek. Teraz wysyła każdą opcję osobno.
oto kod:

[PHP] pobierz, plaintext 
public function action_expiring()
	{ 
 
 
		$announcements = ORM::factory('Announcement')
				->with('user')
				->where(Db::expr('DATEDIFF(announcement.annoucement_availability, NOW())'), '=', 7)
				->find_all();
 
		$i = 0;
 
		foreach ($announcements as $a)
		{
 
				$email = Model_Email::email_by_alias('annoucements_expiring_registered');
 
				$renew_url = Route::url('site_announcements/profile/announcements/renew', array(
					'id' => $a->annoucement_id,
				), 'http');
 
				$email->set_tags(array(
					'%renew_url%' => $renew_url,
					'%renew_link%'	=> HTML::anchor(
						$renew_url, 
						___('announcements.expiring.registered.renew_btn')
					),
					'%annoucement_link%' => HTML::anchor(announcements::url($a, 'http'), $a->annoucement_title),
				));
				$email->send($a->user->user_email);
			}
			$i++;
 
	}
[PHP] pobierz, plaintext 

Nie mam pomysłu jak do tego podejść.

Normalnie, w pętli zbierasz informacje, a poza pętlą wysyłasz mejla...

hmm.. tylko muszę jakoś zebrać te informacje i zapisać je w tablicy i tutaj nie wiem jak to ugryźć.
wystarczy wysyłanie wystawić poza pętle?? czy to byłoby aż takie proste?

Coś takiego:

[PHP] pobierz, plaintext 
$annoucments = =array();
foreach ($announcements as $a) {
    $renew_url = Route::url('site_announcements/profile/announcements/renew', array('id' => $a->annoucement_id,), 'http');
 
    $annoucments[] = array(
        '%renew_url%' => $renew_url, 
        '%renew_link%' => HTML::anchor($renew_url, ___('announcements.expiring.registered.renew_btn')), '%annoucement_link%' => HTML::anchor(announcements::url($a, 'http'), $a->annoucement_title),
    );
}
 
 
    $email = Model_Email::email_by_alias('annoucements_expiring_registered');
    $email->set_tags($annoucments);
    $email->send($a->user->user_email);
[PHP] pobierz, plaintext 

Tylko musisz tą tablicę w szablonie wyrenderować odpowiednio.

Chciałbym pominąć odświeżanie każdej pozycji oddzielnie a przenieść na podstronę wszystkich pozycji. Bardziej mi zależy aby wyświetlić %announcement_link% każdej pozycji czyli odnośnik do każdej pozycji, która spełnia wymagania. Nie wiem czy spójnie logicznie to napisałem.

To ja już nie wiem co Ty chcesz

%announcement_link% to link do ogłoszenia, które należy odświeżyć
wpadłem na pomysł aby zebrać to w 1 email a nie spamować
chciałbym aby w 1 emailu przychodziła lista wszystkich %announcement_link% a nie tak jak teraz każdy email osobno.
Teraz chyba jaśniej napisałem albo mi się tylko tak wydaje

No to przecież dałem Ci rozwiązanie i powiedziałem co musisz jeszcze sam zrobić...

możesz mi przybliżyć jak odpowiednio wyświetlić tą tablice? Męczę się i nic

podbijam proszę o pomoc. w szablonie nie mam możliwości aby wyświetlić tablice muszę zrobić to w tagu %announcement_link%

Jak to kurna nie masz możliwości... To Kochana nie obsługuje iteracji po tablicach w szablonie

zależy mi na tym aby pozostawić funkcjonalność dla ewentualnych moderatorów. Chciałbym aby edycja była prosta poprzed dodanie tagu.

Proszę bardzo o pomoc bo już mi mózg siada.

Ja nie rozumiem o co Ci chodzi...

Czy da się do tagu %announcement_link% przypisać tablice ze wszystkimi tytułami ogłoszeń. Tak aby po wpisaniu tego tagu pojawiły się wszystkie tytuły oddzielone enterem czyli jeden pod drugim ?

Tak, przypisz do tagu %announcement_link% "wszystkie tytuły oddzielone enterem czyli jeden pod drugim".

W tym co Ci dałem

[PHP] pobierz, plaintext 
$annoucments['%annoucement_link%'] = implode('<br>',$annoucments['%annoucement_link%']);
[PHP] pobierz, plaintext 

Wrzucam cały kontroler przetwarzany przez crona czy tak to powinno wyglądać? (nie mam możliwości sprawdzenia bo cron działa raz dziennie o 00:00)

[PHP] pobierz, plaintext 
class Controller_Cron_Announcements extends Controller_Cron_Main {
 
	public function action_expiring()
	{ 
 
 
		$announcements = ORM::factory('Announcement')
				->with('user')
				->where(Db::expr('DATEDIFF(announcement.annoucement_availability, NOW())'), '=', 7)
				->find_all();
 
 
 
		foreach ($announcements as $a)
		{
 
				$renew_url = Route::url('site_announcements/profile/announcements/renew', array('id' => $a->annoucement_id,), 'http');
 
    $annoucments[] = array(
        '%renew_url%' => $renew_url, 
        '%renew_link%' => HTML::anchor($renew_url, ___('announcements.expiring.registered.renew_btn')), 
		'%annoucement_link%' => HTML::anchor(announcements::url($a, 'http'), $a->annoucement_title),
 
    );
	$annoucments['%annoucement_link%'] = implode('<br>',$annoucments['%annoucement_link%']);
	$email = Model_Email::email_by_alias('annoucements_expiring_registered');
    $email->set_tags($annoucments);
    $email->send($a->user->user_email);
			}
}
[PHP] pobierz, plaintext 

Na pewno nie tak. Maila wysyłasz przy każdym przebiegu pętli więc nie będziesz miał wszystkie linków w tablicy. Poza tym fragment z implode (podany przez ~Pyton_000) nie zadziała ($annoucments jest przecież tablicą tablic). Co do niemożności testowania - przecież możesz sobie ręcznie sam wywołać tą akcję kontrolera w taki sam sposób, w jaki robi to cron. Nie musisz czekać do jego automatycznego uruchomienia.

przerobiłbyś mi ten kontroler? nie mam zielonego pojęcia jak do tego podejść

Sam kontroler to jedno, ale widok to drugie. Do widoku powinieneś przekazać całą tablicę $announcements i wyświetlić jej zawartość w pętli (np. foreach). Jak to zrobić w Twoim przypadku? Nie wiem, wygląda na to, że masz jakiś własny system szablonów, albo po prostu robiony jest str_replace na %placeholderach%.

edytowanie email'u mam poprzez zaplecze edytorem cke_editor gdzie używa się tagów ale nie składni php. Masz może jakiś pomysł jak to zrobić?

Przecież ten szablon jest później parsowany przez system szablonów, więc jaki problem napisać jakiegoś foreach po tablicy dookoła tego co już masz?
Przecież to dodanie 2 linijek do szablonu.

O ile to jest system szablonów z jakąś bardziej rozbudowaną funkcjonalnością (warunki, pętle), a nie zwykłe str_replace.

to odpowiada za edytowanie treści email:

[PHP] pobierz, plaintext 
class Form_Admin_Email_Edit extends Bform_Form {
 
	public function  create(array $params = array())
	{
		$this->form_data($params);
 
		$this->add_input_text('email_subject');
 
		$this->add_editor('email_content');
 
		$this->add_input_submit(___('form.save'));
	}
}
[PHP] pobierz, plaintext 

Dlatego chciałem to przypisać w tamtym kontrolerze bo w edytorze nie zadziała kod php

Pokaż kurna szablon w końcu

szablon wygląda tak:

[PHP] pobierz, plaintext 
<h2><?php echo ___('emails.edit.title') ?></h2>
 
<div class="box">
	<?php echo $form ?>
</div>
[PHP] pobierz, plaintext 

jeszcze jest taki główny szablon i nic więcej.

[PHP] pobierz, plaintext 
<h2><?php echo ___('emails.title') ?></h2>
 
<div class="box">
 
	<div class="table_filters clearfix">
		<?php echo Form::open(NULL, array('method' => 'GET', 'id' => 'show_emails_form', 'class' => 'pull-right bform form-horizontal')) ?>
		<div class="control-group">
			<?php echo Form::label('type', ___('emails.filters.types').': ') ?>
			<div class="controls">
				<?php echo Form::select('type', Arr::unshift($types, NULL, ''), Arr::get($filters, 'type')); ?>
			</div>
		</div>
		<?php echo Form::submit(NULL, ___('form.show'), array('class' => 'btn')) ?>
		<?php echo Form::close() ?>
	</div>
 
	<hr/>
 
	<table class="table tablesorter">
		<thead>
			<tr>
				<th><?php echo ___('email_alias') ?></th>
				<th><?php echo ___('email_description') ?></th>
				<th><?php echo ___('admin.table.actions') ?></th>
			</tr>
		</thead>
		<tbody>
			<?php foreach ($emails as $e): ?>
				<tr>
					<td><?php echo $e->email_alias ?></td>
					<td><?php echo $e->email_description ?></td>
					<td>
						<a href="<?php echo url::site('/admin/emails/edit/' . $e->email_id) ?>"><?php echo ___('admin.table.edit') ?></a>
					</td>
				</tr>
			<?php endforeach ?>
		</tbody>
	</table>
</div>
[PHP] pobierz, plaintext 

Nie wiem jak to działa ale w emailu wysyła się tylko to co wpiszę w cke_editor

proszę o pomoc jak to ugryźć.

Szablon mejla + ew. kontroler który obsługuje tą wysyłkę.

kontroler podałem na samym początku a szablon to chyba to co jest wyżej bo więcej nic nie mam.

Czyli to co podłeś wpisujesz w CKEditor ?

w ck_editor wpisuję tylko treść meila i email tag czyli np %renew_url% lub %announcement_link%
masz może jakiś pomysł jak to zrobić?

Proszę o pomoc zanim zniknę w milionie tematów.

dawaj jeszcze Model_Email

[PHP] pobierz, plaintext 
public function send_email_message($subject, $message = NULL, $params = NULL)
	{
		if($subject instanceof Model_Email)
		{
			return $subject->send($this->get_email_address(), $params);
 
		}
 
		return email::send($this->get_email_address(), $subject, $message, $params);
	}
[PHP] pobierz, plaintext 

To jest cała klasa ?

nie tylko metoda odpowiedzialna za email. cała klasa jest większa ale dotyczy czego innego.

I pewnie w tej klasie masz też podstawianie parametrów do %placeholderów%. Ale skoro uważasz, że to co podajesz jest wystarczające to może lepiej od razu zamknąć wątek, bo w końcu ktoś nie wytrzyma i wybuchnie, a Ty i tak pomocy nie uzyskasz.

Tak masz racje tylko w innej metodzie to było

[PHP] pobierz, plaintext 
public function approve()
	{
		$this->annoucement_is_approved = 1;
		$this->save();
 
		$email = ORM::factory('Email')
				->where('email_alias', '=', 'annoucement_approved')
				->find();
 
		$url = announcements::url($this, 'http');
 
		$email->set_tags(array(
			'%link%'	=> ('<a href="' . $url . '">' . $url . '</a>'),
		));
 
		if ($this->user->user_id)
		{
			$email->send($this->user->user_email);
		}
		else
		{
			$email->send($this->annoucement_email);
		}
	}
[PHP] pobierz, plaintext 

Słuchaj przepraszam jeżeli komuś podniosłem ciśnienie.

[PHP] pobierz, plaintext 
class Controller_Cron_Announcements extends Controller_Cron_Main {
 
	public function action_expiring()
	{ 
 
 
		$announcements = ORM::factory('Announcement')
				->with('user')
				->where(Db::expr('DATEDIFF(announcement.annoucement_availability, NOW())'), '=', 7)
				->find_all();
 
		$i = 0;
 
		foreach ($announcements as $a)
		{
			if ($a->user_id) 
			{
				$email = Model_Email::email_by_alias('annoucements_expiring_registered');
 
				$renew_url = Route::url('site_announcements/profile/announcements/renew', array(
					'id' => $a->annoucement_id,
				), 'http');
 
				$email->set_tags(array(
					'%renew_url%' => $renew_url,
					'%renew_link%'	=> HTML::anchor(
						$renew_url, 
						$a->annoucement_title
					),
					'%annoucement_link%' => HTML::anchor(announcements::url($a, 'http'), $a->annoucement_title),
				));
				$email->send($a->user->user_email);
			}
			elseif ($a->annoucement_email) 
			{
				$email = Model_Email::email_by_alias('annoucements_expiring_not_registered');
 
				$renew_url = Route::url('site_announcements/frontend/announcements/renew', array(
					'id' => $a->annoucement_id,
					'token' => $a->annoucement_hash,
				), 'http');
 
				$email->set_tags(array(
					'%renew_url%' => $renew_url,
					'%renew_link%'	=> HTML::anchor(
						$renew_url, 
						___('announcements.expiring.not_registered.renew_btn')
					),
					'%add_link%'	=> HTML::anchor(
						Route::url('site_announcements/frontend/announcements/add', NULL, 'http'),
						___('announcements.expiring.not_registered.add_btn')
					),
					'%annoucement_link%' => HTML::anchor(announcements::url($a, 'http'), $a->annoucement_title),
				));
				$email->send($a->annoucement_email);
			}
			$i++;
		}
	}
 
}
 
[PHP] pobierz, plaintext 

@dominick wybacz że to powiem, ale idź do domu i nie wracaj.

Ja Cię prosiłem o kod Klasy, a Ty swoje. Nie chcesz pomocy to trzeba było powiedzieć że masz w 4 literach to co ktoś do Ciebie mówi. Ja Ci nie pomogę, straciłem cierpliwość, nie ma co się wysilać jeśli i tak tego nie rozumiesz. Ulżyło mi...

Ok nawrzucałeś mi to teraz może zaczniemy od początku wrzucę całą klasę.

[PHP] pobierz, plaintext 
class Model_Email extends ORM {
 
	protected $_table_name = 'emails';
	protected $_primary_key = 'email_id';
 
	public function find_by_alias($alias)
	{
		return $this->where('email_alias', '=', $alias)->find();
	}
 
	public function set_tags(array $tags = array())
	{
		foreach ($tags as $pattern => $replacement)
		{
				$this->email_subject = str_replace($pattern, $replacement, $this->email_subject);
				$this->email_content = str_replace($pattern, $replacement, $this->email_content);
		}
		return $this;
	}
 
	public function send($to, $params = array())
	{
		return email::send($to, $this->email_subject, $this->email_content, $params);
	}
 
	public static function email_by_alias($alias)
	{
		$self = new self();
		$self->where('email_alias', '=', $alias)
			->find();
 
		if(!$self->loaded())
		{
			throw new Kohana_Exception('Cannot find e-mail alias :alias', array(':alias' => $alias));
		}
 
		return $self;
	}
}
[PHP] pobierz, plaintext 

niestety zwykłe str_replece.
Słuchajcie przepraszam, że tak ciężko było mi dotrzeć do tego. Problem z czytaniem ze zrozumieniem ale pracuje nad tym.

Proszę o pomoc nadal

Pokaż treść mejla którą wpisujesz

[HTML] pobierz, plaintext 
Witaj,
twoje ogłoszenia wymagają odświeżenia:
%renew_link%
 
 
Aby to zrobić kliknij na link poniżej, który przeniesie Cie do Twoich ogłoszeń.
%renew_url%
[HTML] pobierz, plaintext 

To wszystko co wpisuje w email


proszę o pomoc

Nikt nie wiem co z tym zrobić?

[PHP] pobierz, plaintext 
public function set_tags(array $tags = array())
	{
		foreach ($tags as $pattern => $replacement)
		{
				if(is_array($replacement)) $replacement = implode(', ', $replacement);
 
				$this->email_subject = str_replace($pattern, $replacement, $this->email_subject);
				$this->email_content = str_replace($pattern, $replacement, $this->email_content);
		}
		return $this;
	}
[PHP] pobierz, plaintext 

I teraz pseudokod:

[PHP] pobierz, plaintext 
$ogloszenia = pobierz_te_ktore_wymagaja_odswierzenia();
 
$links = array();
 
foreach($ogloszenia as $ogloszenie) {
$links[] = HTML::anchor($ogloszenie['link], $ogloszenie['nazwa']);
}
 
$mail = new Model_Mail('ktos@cos.com', 'jakis-szablon', 'etc');
$mail->setTags('%renew_link%', $links);
$mail->send();
[PHP] pobierz, plaintext 

Chodzi o sam sens, po skopiowaniu nie zadziała

Dziękuje biore się za to bo już mi czaszka zadymiła ; )

Skleiłem coś takiego:

[PHP] pobierz, plaintext 
class Controller_Cron_Announcements extends Controller_Cron_Main {
 
	public function action_expiring()
	{ 
 
 
		$announcements = ORM::factory('Announcement')
				->with('user')
				->where(Db::expr('DATEDIFF(announcement.annoucement_availability, NOW())'), '=', 7)
				->find_all();
 
				$links = array();
		foreach ($announcements as $a) 
		{
    $renew_url = Route::url('site_announcements/profile/announcements/renew', array('id' => $a->annoucement_id,), 'http');
	$links[] = HTML::anchor($renew_url, ___('announcements.expiring.registered.renew_btn')),$a->annoucement_title);
		}
 
 
    $email = Model_Email::email_by_alias('annoucements_expiring_registered');
    $email->set_tags('%renew_link%',$links);
    $email->send($a->user->user_email);
		}
}
 
 
[PHP] pobierz, plaintext 

Czy jest ok? niestety serwer na którym mam stronę odpala cron raz dziennie o 00:00

Udało mi się skontaktować z serwerownią i uruchomić cron co 5 minut jednak skrypt nie działa

proszę o pomoc

A jaki masz problem żeby odpalić polecenie ręcznie to które masz zapisane w cronie ?

I nie działa bo??

Już cron zrobiłem jednak nie wysyła meila wogóle.

Jak mam dojść do rozwiązania problemu?


Nikt już nic nie wymyśli?

wchodze tu codziennie i czekam na odpowiedź bo sam wiem, że nie dam rady gdyz męczyłem się z miesiąc. Niestety chyba na marne.

Wiesz że jesteś upierdliwy?

Co do problemu:

Za dużo o 1 )

[PHP] pobierz, plaintext 
$email->send($a->user->user_email);
[PHP] pobierz, plaintext 

Jest poza foreach więc nie ma zmiennej $a więc nie ma gdzie wysłać...

Nie wiem czy dobrze myślę:
mam przypisać w pętli foreach do zmiennej np. $email_user = $a->user->user_email
a potem poza pętlą użyć zmiennej $email_user?

Powiedzmy że tak.
Tylko że wtedy jeżeli wygasające ogłoszenia są różnych użytkowników to wszystkie pójdą do tego który miał ostatnie na liście.

czyli zapisać do tabilcy? Tylko nie wiem jak z wysyłaniem tego email'u gdy wszystkie będą w tablicy


dobrze kombinuje?

Musisz:
- Pobrać wszystkie ogłoszenia (to już masz)
- Pogrupować ogłoszenia wg. adresu email
- dla każdej grupy email wygenerować zbiorowy link
- Wysłać ten link dla adresu grupy.

To mnie teraz zaskoczyłeś... Nie wiem jak się zabrać i co znaczy pogrupować

A znasz Ty się w ogóle na PHP czy tylko udajesz że coś umiesz?