Discussion :

[Lharnisch]

Bonjour à tous,

Pour un site ECommerce, je cherche à intégrer 2 variables à un numéro de téléphone commençant par 555 dans un mail de confirmation de commande :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

echo '<p style="padding-bottom: 2em;">Si vous possédez un Smartphone : <a href="tel:555'.$carte->scnum.$carte->pin.'">ACTIVEZ AUTOMATIQUEMENT VOTRE RECHARGE</a> (Appel gratuit)</p>';

Avec cette ligne, quand on clique sur le lien, l'appel se lance mais compose uniquement le 555, comment faire pour qu'il compose les numéros suivants correspondants à ces 2variables collées l'une à l'autre ?

555+variablescnum+variablepin

Merci pour votre aide :-)

[Invité]

Bonjour,

As-tu au moins vérifié le code html généré ?

[Lharnisch]

Merci pour ton message.

Dans le mail concerné, l'affichage de la phrase est bon, c'est le lien qui ne va pas.
J'espère avoir répondu à ta question.

[sabotage]

Est-ce que le numéro entier est bien présent dans le lien ?

[Lharnisch]

Non dans le lien il n'y a que "tel:555" (sans les guillemets)
Si je clique dessus l'appel se lance mais ne compose que 555

[Invité]

C'est donc que $carte->scnum et $carte->pin sont vides.

Maintenant, tu sais quoi chercher.

[Lharnisch]

Donc ça ne vient pas d'une erreur de syntaxe ? guillemets au mauvais endroit, etc... ?

[Lharnisch]

Pourtant elles sont réutilisées juste après et ça fonctionne... ligne 25 :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
echo '<p style="padding-bottom: 2em;">Si vous possédez un Smartphone : <a href="tel:555'.$carte->scnum.$carte->pin.'">ACTIVEZ AUTOMATIQUEMENT VOTRE RECHARGE</a> (Appel gratuit)</p>';
 
  echo '<style type="text/css">
.tg  {border-collapse:collapse;border-spacing:0;}
.tg td{font-family:Arial, sans-serif;font-size:14px;padding:10px 5px;border-style:solid;border-width:0px;overflow:hidden;word-break:normal;}
.tg th{font-family:Arial, sans-serif;font-size:14px;font-weight:normal;padding:10px 5px;border-style:solid;border-width:0px;overflow:hidden;word-break:normal;}
.tg-032e{text-align:center;}
@media screen and (max-width: 767px) {.tg {width: auto !important;}.tg col {width: auto !important;}.tg-wrap {overflow-x: auto;-webkit-overflow-scrolling: touch;}}</style>
<div class="tg-wrap"><table class="tg" style="margin:0 auto;">';
  foreach($thecartes as $carte){
    $product = commerce_product_load_by_sku($carte->RCHGP_ID);
    $img =file_create_url($product->field_image['und'][0]['uri']);
 
    ?>
    <tr>
    <th class="tg-031e" style="text-align:center;border-bottom:1px solid #D4D4D4 !important;">
        <?php echo '<img style="width:40%;max-width:400px;padding:1em;" src="'.$img.'" alt="'.$product->title.'">'; ?>
      </th>
      <th class="tg-032e" style="text-align:left;border-bottom:1px solid #D4D4D4 !important;">
        <?php
        $date = date_create_from_format('Ymd', $carte->end_date);
        $dateform = date_format($date,'d/m/Y');
        $plus = isset ($product->field_offre['und'][0]['value']) ? ' '.$product->field_offre['und'][0]['value'] : '';
        echo '<p style="color: #ca243b;"><strong>"'.$product->title.' F CFP HT'.$plus.'"</strong></p>';
        echo '<p>Votre code de rechargement est le : <strong>'.$carte->scnum.$carte->pin.'</strong>.</p>';
        echo '<p>Sa date limite d\'initialisation est le : <strong>'.$dateform.'</strong>.</p>';
        echo '<p>Le numéro de série de cette Recharge Liberté est le : <strong>'.$carte->senum.'</strong>.</p>';

[Invité]

Sauf que tu es en dehors du :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

  foreach($thecartes as $carte){

N.B. Tu devrais mieux indenter ton code : il serait plus facile à lire et à déboguer.
Et sortir le HTML du PHP.

ex. au lieu de :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
<?php
echo '<p style="padding-bottom: 2em;">Si vous possédez un Smartphone : <a href="tel:555'.$carte->scnum.$carte->pin.'">ACTIVEZ AUTOMATIQUEMENT VOTRE RECHARGE</a> (Appel gratuit)</p>';

écrire :

Code mixte :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
<?php
// on ferme PHP
?>
   <p style="padding-bottom: 2em;">Si vous possédez un Smartphone : <a href="tel:555<?php echo $carte->scnum.$carte->pin; ?>">ACTIVEZ AUTOMATIQUEMENT VOTRE RECHARGE</a> (Appel gratuit)</p>
<?php // reprise du PHP

Ne serait-ce que pour la coloration syntaxique (en tout cas sur Notepad++ par ex.), qui permet aussi de déboguer plus facilement.

[Lharnisch]

OK je viens de comprendre, merci mille fois !

Au final, si je veux laisser mon texte au même endroit, est ce que je dois dupliquer le Foreach avant celui-ci ?
J'imagine qu'il y a un moyen plus simple ;-)

[Invité]

Si tu as un foreach, c'est qu'il y en a plusieurs.

A toi de (sa)voir si c'est toujours le même pour tous, ou un différent pous chacun.


Et comme je te l'ai conseillé, même dans une fonction, inutile de mettre des echo à tire-larigot : sort le HTML du PHP.

[Lharnisch]

Oui je vais modifier ça, j'ai récupéré ce module en l'état et n'étant pas développeur je galère à le faire évoluer...

Merci pour ces conseils !