<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 11, 2015 at 1:41 PM, Daniel Depaoli <span dir="ltr"><<a href="mailto:daniel.depaoli@create-net.org" target="_blank">daniel.depaoli@create-net.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Hi all.<div>I'm starting to resolve the todo at these line[1]. To solve this I think to hardcoded the role in the file, for example:</div><div><br></div><blockquote style="margin:0px 0px 0px 40px;border:none;padding:0px"><div><i>$swift_proxies = get_nodes_hash_by_roles($network_metadata, ['primary-swift-proxy', 'swift-proxy']) ? {</i></div><div><i>    true => get_nodes_hash_by_roles($network_metadata, ['primary-swift-proxy', 'swift-proxy']),</i></div><div><i>    false => get_nodes_hash_by_roles($network_metadata, ['primary-controller', 'controller']),</i></div><div><i>}</i></div></blockquote><br>Is this the right way or do you suggest something more clean?</div></blockquote></div><div class="gmail_extra"><br></div>Function <i>get_nodes_hash_by_roles returns </i>hash<i>. </i>Provided above switch shouldn't work. But this way is true. </div><div class="gmail_extra">IMHO list of roles name should be constructed before.<br><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><br><div>/sv</div></div></div></div>
</div></div>