Discussion :

[mout1234]

Bonjour,

M'inspirant d'un code existant (je suis admin et je débute en développement), je souhaite ajouter un bouton à la page Compte pour créer une nouvelle opportunité liée à ce compte.

Le code semble fonctionner mais il génère un message d'erreur lorsque je teste la classe System.NullPointerException: Attempt to de-reference a null object.
L'erreur se situe à la ligne vf.init() de la méthode de test.

Avez-vous une idée de ce qui pose problème?

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
28
29
30
31
32
33
34
35
36
37
38
39
public with sharing class New_Opportunity_On_Account
{
  private string AccountId;
  
  public New_Opportunity_On_Account ()
  {
    AccountId = ApexPages.currentPage().getParameters().get('id');
  }
  
  public Pagereference init()
  {        
      
    PageReference p = new PageReference(Constants.RECORD_TYPE_URL);
    p.getParameters().put('ent','Opportunity');
    p.getParameters().put(constants.RET_URL,'/'+AccountId); 
    string save_Url = Constants.OPPORTUNITY_EDIT_URL + '?nooverride=1&';
    
    save_Url +=  constants.ACCOUNT_ID_FIELD_ID + '=' + AccountId  + '&' +
        constants.RET_URL + '=' + '/'+AccountId;
    
    p.getParameters().put(Constants.SAVE_NEW_URL,save_Url);
    
    return p;
  }
  
  /************************************TEST************************************/
  public static testMethod void test()
  {
    Account acc = New Account();
    acc.Name= ... quelques données renseignées
    insert acc;
    PageReference p  = Page.New_Opportunity_On_Account;
    p.getParameters().put('id',acc.id);
    Test.setCurrentPageReference(p);
    New_Opportunity_On_Account vf = new New_Opportunity_On_Account();
    vf.init();
  }

}

La page Visualforce est rudimentaire

Code :

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

1
2
3
<apex:page controller="New_Opportunity_On_Account" action="{!init}">

</apex:page>

[v.charlet]

je dirais d'instinct que ApexPages.currentPage() retourne NULL dans le constructeur de New_Opportunity_On_Account

En regardant mon code, j'ai utilisé :
System.currentPageReference().getParameters().get('Id');
pour récupérer les paramètres

Fais l'essai en le remplaçant dans le constructeur et le test, peut être que la class ApexPages ne fonctionne que dans le context Salesforce et pas en tests

Remarque: j'aime pas trop la methodtest écrite, le fait de créer la page va aussi créer le controlleur que tu crée à nouveau ... c'est douteux, mais le problème ne vient pas forcément de là

[mout1234]

je dirais d'instinct que ApexPages.currentPage() retourne NULL dans le constructeur de New_Opportunity_On_Account

En regardant mon code, j'ai utilisé :
System.currentPageReference().getParameters().get('Id');
pour récupérer les paramètres

Fais l'essai en le remplaçant dans le constructeur et le test, peut être que la class ApexPages ne fonctionne que dans le context Salesforce et pas en tests

Remarque: j'aime pas trop la methodtest écrite, le fait de créer la page va aussi créer le controlleur que tu crée à nouveau ... c'est douteux, mais le problème ne vient pas forcément de là

Bonjour

Merci Vincent pour ton conseil.

Je viens de tester rapidement en changeant dans le constructeur mais cela ne règle pas le problème.

Je ne vois pas comment je pourrai l'appliquer dans la fonction de test comme tu le suggères et suis partant pour revoir la structure de la fonction test mais ... je n'ai pas encore les compétences

Ce qui m'intrigue c'est que je me suis inspiré d'un autre code qui lui ne génère pas de message d'erreur. J'en déduis que j'ai - comme souvent - le problème sous les yeux sans m'en rendre compte

[v.charlet]

le code fonctionne mais pas le test c'est bien ça ?

peut être voir du coté de Test.setcurrentPage(pageReference page) ??
http://www.salesforce.com/us/develop...inName=webhelp

il y a la dernière réponse de ce thread qui donne un exemple plutôt clair d'une procédure de test :
http://boards.developerforce.com/t5/...ss/td-p/136418

[Overcrash]

Bonjour,

C'est quelque chose que tu as envie de faire ?
Car la liaison existe déjà pour créer une opportunité rattaché au compte.

[mout1234]

Bonjour,

C'est quelque chose que tu as envie de faire ?
Car la liaison existe déjà pour créer une opportunité rattaché au compte.

Bonjour,

Je n'ai pas repéré cette possibilité, à part via la liste associée d'opportunités que je ne peux utiliser dans notre cas (en raison d'autres personnalisations).

Il y a un autre endroit?j'ai raté quelque chose ???

Ceci dit, j'ai de toute façon une commande similaire à mettre en place pour la création d'un objet personnalisé lié au compte.

[mout1234]

Bonjour,

Pour information, j'ai un peu avancé: en tâtonnant, il semble que le problème vienne de l'étape intermédiaire de sélection de type d'enregistrement.
J'ai revu le code comme ceci

Code :

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

1
2
3
4
5
6
7
 public Pagereference init()
  {        
    string save_url ='/006/e?nooverride=1&opp4_lkid=001R000000iv3fc +'&' +
                    constants.CLOSED_DATE_FIELD_ID +'='
                        +constants.CLOSED_DATE +'&'+constants.RET_URL +'=/'+AccountId;  
    PageReference p = new PageReference(save_url);
return p;

et je n'ai plus d'erreur.
Ceci dit, je ne trouve pas cela très clean et je ne comprends pas pourquoi le code précédent pose problème alors qu'il fonctionne sans souci dans une autre classe.

[Overcrash]

Je regarde demain je te fais un retour.

[tomasi]

Bonjour,

Il y a un moyen très simple de faire ça sans aucun code, il suffit de mettre le lien suivant :

Code :

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

/006/e?accid={!Account.Id}&opp3={!Account.Name}

sinon une méthode encore plus simple est d'utiliser le bouton nouvelle opportunité sur la liste associée opportunité affichée sur le compte.

[mout1234]

Bonjour,

Il y a un moyen très simple de faire ça sans aucun code, il suffit de mettre le lien suivant :

Code :

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

/006/e?accid={!Account.Id}&opp3={!Account.Name}

sinon une méthode encore plus simple est d'utiliser le bouton nouvelle opportunité sur la liste associée opportunité affichée sur le compte.

Bonsoir tomasi,

Désolé pour ma réponse tardive.
J'ai essayé ce que tu proposes. Une opportunité est bien créée mais le champ Account n'est pas alimenté... Je suis occupé par autre chose en ce moment et je n'ai pas cherché en détail ce qui ne va pas...

Le bouton dans la liste associée n'est pas applicable dans mon cas car nous n'affichons pas cette liste pour d'autres raisons (gestion de droits non standard...).

Je reviendrai dans cette discussion lorsque je me replongerai dans ce bout de code et - j'espère - que je trouverai enfin où est le problème.

Merci à tous.