Produits Entr'ouvert » Hobo

• pour combo, son slug c'est portal
• pour A2 avec la ville comme Publik principal, c'est _hobo-tm_portal

Le paramètre ?service= accepter comme syntaxe "legacy" de sa valeur "{service.slug}" mais pour être sûr de son coup il faudrait normalement "{ou.slug} {service.slug}".

Je vois bien une variable ou-slug mais je ne sais pas si elle est défini sur le hobo primaire ou le secondaire... je regarde ça.

Je teste et ça me produit ?service=idp (sur le combo primaire) et ?service=_hobo-commune2__interco_idp sur un combo secondaire.

Je dirais que le soucis c'est slug = service['slug'] fait qu'on a le slug de l'idp pas le slug de l'application sur laquelle on est.

Bon ok en fait c'est déjà partiellement fait via portal_user_slug, c'est juste que ça ne prend pas en compte le cas primaire/secondaire.

le portal user ne peut pas être secondaire

? c'est commun d'avoir dans les publik secondaires des portail usager et portail agent spécifiques.

_~~

(je trouve que les f-strings ici dégradent la lisibilité)

Frédéric Péters a écrit :

le portal user ne peut pas être secondaire

? c'est commun d'avoir dans les publik secondaires des portail usager et portail agent spécifiques.

Tu m'as mal compris, je veux dire que dans le code suivant :

            if service.get('service-id') == 'combo' and not service.get('secondary'):
                if 'portal-user' in service.get('template_name', ''):
                    variables['portal_user_url'] = service.get('base_url')
                    variables['portal_user_title'] = service.get('title')
                    variables['portal_user_slug'] = service.get('slug')
...
        if 'idp_registration_url' in variables:
            params = {}
            if 'portal_user_url' in variables:
                params['next'] = variables['portal_user_url']
            if 'portal_user_slug' in variables:
                # if we are in a secondary hobo, adapt the ou slug and
                # portal_user_slug to match the service slug in the Authentic
                # of the primary hobo
                if authentic_service.get('secondary'):
                    ou_slug = variables['ou-slug']
                    service_slug = '_%s_%s' % (ou_slug, variables['portal_user_slug'])
                    params['service'] = '%s %s' % (ou_slug, service_slug)
                else:
                    # we should provider the default slug to have a full
                    # service reference, but it could change so for now we
                    # expect authentic to search first in the default ou
                    params['service'] = variables['portal_user_slug']

J'ai juste à vérifier que l'authentic est secondaire, le service ayant produit la variable 'portal_user_slug' est forcément primaire pour le service en cours (celui qui appelle le settings_loader TemplateVars) vu la toute première condition (dans mon patch précédent je retestais).

_~~

(je trouve que les f-strings ici dégradent la lisibilité)

Ok je corrige ça.

Ça casse toutes les pages combo ou il y a une page faisant redirection vers {{ idp_url }}accounts/.

Il faut à priori substituer par la nouvelle variable créée {{ idp_account_url }}.

Et documenter quelque part.

+                variables['idp_account_url'] += '?%s' % urlencode(params)

{{idp_url}}accounts/edit/

{{idp_account_url}}edit/

Valentin Deniaud a écrit :

[...]
or dans ce que pointe Serghei plus haut
[...]
ce qui nous met la puce à l'oreille que quelque part il peut exister
[...]
Bref, virer cette ligne, en bonus on conserve la cohérence avec le comportement de {{idp_url}}.

Si on vire cette ligne le patch n'a plus aucun intérêt, le but étant d'avoir des URLs avec ce paramètre ?service= dans les entête de page (lien page mon compte), de ce que je vois dans publik-base-theme il n'y aucune utilisation autre que directe :

templates/includes/user-info.html:    {% if idp_account_url %}<a class="account-link" href="{{idp_account_url}}">{% endif %}
templates/includes/user-info.html:    <span class="connected-user">{% block user-info-user-name %}{{user.first_name}} {{user.last_name}}{% endblock %}</span>{% if idp_account_url %}</a>{% endif %}
templates/variants/metz-metropole-2019/includes/user-info.html:  <a class="hello" href="{{idp_account_url}}"><span>Bonjour {{ user.first_name }}</span></a><span> -
templates/variants/strasbourg-2018/combo/page_template.html:        <a href="{{ idp_account_url}}" class="nav-account nav-btn" title="Mon compte">
templates/variants/toodego/wcs/base.html:    {% if idp_account_url %}<a href="{{idp_account_url}}">{% endif %}
templates/variants/toodego/wcs/base.html:    <span class="connected-user">{{session_user_display_name}}</span>{% if idp_account_url %}</a>{% endif %}
templates/variants/toulouse-metropole/includes/nav.html:            {% if idp_account_url %}<a href="{{idp_account_url}}">{% endif %}
templates/variants/toulouse-metropole/includes/nav.html:            <span class="connected-user">{{user.first_name}} {{user.last_name}}</span>{% if idp_account_url %}</a>{% endif %}
templates/variants/villeneuve-dascq/includes/user-info.html:    {% if idp_account_url %}<a class="account-link" href="{{idp_account_url}}">{% endif %}
templates/variants/villeneuve-dascq/includes/user-info.html:    <span class="connected-user">{% block user-info-user-name %}{{user.first_name}} {{user.last_name}}{% endblock %}</span>{% if idp_account_url %}</a>{% endif %}
templates/variants/villeurbanne-2018/user_info.html:  {% if idp_account_url %}<a href="{{idp_account_url}}" role="button" class="btn-account">{% endif %}
templates/variants/villeurbanne-2018/user_info.html:  Mon compte{% if idp_account_url %}</a>{% endif %}

Il n'y qu'idp_url qui est utilisé de cette manière.

Benjamin Dauvergne a écrit :

le but étant d'avoir des URLs avec ce paramètre ?service= dans les entête de page (lien page mon compte)

Ça me paraît clair que du coup il faut aller explicitement toucher cet/ces endroit(s) et y ajouter ?{{idp_service_and_next_params}}.

Automatic expiration