Est ce la bonne méthode pour remplir un contenu particulier

Il y a plusieurs façon de mettre en place un workflow dans Drupal, donc ce serait sympa de préciser quel module tu utilises dans ta question ;)

Ceci dit vu les requêtes SQL j'en déduit que tu utilises le module Workflow :P

Ma réponse sera d'abord : est-il pertinent d'écrire un module custom alors que tu peux faire ce que tu souhaites avec Views ?

D'ailleurs le module Workflow créé deux vues par défaut workflow_summary et workflow_pending. Après, il est assez facile d'ajouter un filtre (ou un argument, en fonction de ce que tu veux faire) par rôle...

(*) Ok Yoran et certains codeurs ne seront pas forcémment - non sans raison - d'accord avec moi ici et il te donnera probablement une méthode fondée sur l'API, sans être codeur je pense que ton code ne tire pas parti de l'API de Drupal (cf http://api.drupal.org/).

@David ROFL Bon ok je dis plus rien moi :)))) Tu sais je crois que j'ai jamais fait un seul site qui utilise views, et personne n'est mort ;-)

Mais chacun son truc, moi ce qui m'agace en revanche, c'est ceux qui vendent à leur prospect un views qui 'fiabilise leur investissement' et toi tu repasses derrière, et tu te fais "jeter" car tu proposes de bosser "proprement"...

D'ailleurs faudra qu'on m'explique la rentabilisation de l'investissement de Views1 sur Views2... ;-)

@tff

Ton code est juste, et devrait fonctionner, juste quelques remarques :
1/ Tu serais pas javaiste toi dés foi ? ;-) En php tu peux utiliser les multi-line strings, c'est bien pratique.
2/ Penses à utiliser les %d ou %s pour passer des paramètres à tes requêtes. C'est plus joli et en plus cela prend en charge un peu de sécurité (échappement des chaînes par ex. )
3/ Le db_rewrite_sql ne fonctionnera (il me semble) que si tu aliases ta table node avec n (node as n). Et du coup tu ne devrait pas être obligé d'ajouter dans ta requête la partie "access".
4/ Pour être certain que ta requete est bonne, david à raison, tu peux utilisre Views2... pour générer la requête et tu jetes le reste ;-p.

Sinon, seule remarque

<?php
$args1 = array();
global $user;
$sql = "
 SELECT 
        node.title, 
       node.nid,
      workflow_access.rid,
       workflow_states.
       weight,
        users_roles.uid,
       workflow_access.grant_view 
    FROM node 
 INNER JOIN workflow_node ON workflow_node.nid = node.nid
   INNER JOIN workflow_access ON workflow_access.sid = workflow_node.sid
  INNER JOIN users_roles ON users_roles.rid = workflow_access.rid
    INNER JOIN workflow_states ON workflow_states.sid = workflow_access.sid
    WHERE 
     workflow_access.grant_view=1 AND
       workflow_states.weight<10 AND
       users_roles.uid = %d";

//$sql = db_rewrite_sql($sql);
$cursor = db_query($sql, $user->uid);

....
?>