]> git.mxchange.org Git - friendica.git/commitdiff
Move poke module to src/
authorHypolite Petovan <hypolite@mrpetovan.com>
Mon, 20 Apr 2020 15:42:27 +0000 (11:42 -0400)
committerHypolite Petovan <hypolite@mrpetovan.com>
Mon, 20 Apr 2020 15:47:08 +0000 (11:47 -0400)
- Add new route contact/<cid>/poke and update all references
- Rework template with form field includes
- [frio] Enable modal behavior

12 files changed:
include/conversation.php
mod/poke.php [deleted file]
src/Model/Contact.php
src/Module/Contact/Poke.php [new file with mode: 0644]
static/routes.config.php
view/global.css
view/templates/contact/poke.tpl [new file with mode: 0644]
view/templates/poke_content.tpl [deleted file]
view/templates/poke_head.tpl [deleted file]
view/theme/frio/css/style.css
view/theme/frio/js/theme.js
view/theme/frio/theme.php

index 0ae8dd14a129f3f64cde7d3ef1cfb4e19968a730..43eeb9e41ce1cced73d39bdb72e3ee27e7945303 100644 (file)
@@ -837,15 +837,15 @@ function item_photo_menu($item) {
 
        if (!empty($pcid)) {
                $contact_url = 'contact/' . $pcid;
-               $posts_link = 'contact/' . $pcid . '/posts';
-               $block_link = 'contact/' . $pcid . '/block';
-               $ignore_link = 'contact/' . $pcid . '/ignore';
+               $posts_link  = $contact_url . '/posts';
+               $block_link  = $contact_url . '/block';
+               $ignore_link = $contact_url . '/ignore';
        }
 
        if ($cid && !$item['self']) {
-               $poke_link = 'poke?c=' . $cid;
                $contact_url = 'contact/' . $cid;
-               $posts_link = 'contact/' . $cid . '/posts';
+               $poke_link   = $contact_url . '/poke';
+               $posts_link  = $contact_url . '/posts';
 
                if (in_array($network, [Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA])) {
                        $pm_url = 'message/new/' . $cid;
diff --git a/mod/poke.php b/mod/poke.php
deleted file mode 100644 (file)
index a683b11..0000000
+++ /dev/null
@@ -1,191 +0,0 @@
-<?php
-/**
- * Poke, prod, finger, or otherwise do unspeakable things to somebody - who must be a connection in your address book
- * This function can be invoked with the required arguments (verb and cid and private and possibly parent) silently via ajax or
- * other web request. You must be logged in and connected to a profile.
- * If the required arguments aren't present, we'll display a simple form to choose a recipient and a verb.
- * parent is a special argument which let's you attach this activity as a comment to an existing conversation, which
- * may have started with somebody else poking (etc.) somebody, but this isn't necessary. This can be used in the more pokes
- * addon version to have entire conversations where Alice poked Bob, Bob fingered Alice, Alice hugged Bob, etc.
- *
- * private creates a private conversation with the recipient. Otherwise your profile's default post privacy is used.
- *
- * @file mod/poke.php
- */
-
-use Friendica\App;
-use Friendica\Core\Hook;
-use Friendica\Core\Logger;
-use Friendica\Core\Renderer;
-use Friendica\Core\System;
-use Friendica\Database\DBA;
-use Friendica\DI;
-use Friendica\Model\Item;
-use Friendica\Protocol\Activity;
-use Friendica\Util\Strings;
-use Friendica\Util\XML;
-
-function poke_init(App $a)
-{
-       if (!local_user()) {
-               return;
-       }
-
-       $uid = local_user();
-
-       if (empty($_GET['verb'])) {
-               return;
-       }
-
-       $verb = Strings::escapeTags(trim($_GET['verb']));
-
-       $verbs = DI::l10n()->getPokeVerbs();
-
-       if (!array_key_exists($verb, $verbs)) {
-               return;
-       }
-
-       $activity = Activity::POKE . '#' . urlencode($verbs[$verb][0]);
-
-       $contact_id = intval($_GET['cid']);
-       if (!$contact_id) {
-               return;
-       }
-
-       $parent = (!empty($_GET['parent']) ? intval($_GET['parent']) : 0);
-
-
-       Logger::log('poke: verb ' . $verb . ' contact ' . $contact_id, Logger::DEBUG);
-
-
-       $r = q("SELECT * FROM `contact` WHERE `id` = %d AND `uid` = %d LIMIT 1",
-               intval($contact_id),
-               intval($uid)
-       );
-
-       if (!DBA::isResult($r)) {
-               Logger::log('poke: no contact ' . $contact_id);
-               return;
-       }
-
-       $target = $r[0];
-
-       if ($parent) {
-               $fields = ['uri', 'private', 'allow_cid', 'allow_gid', 'deny_cid', 'deny_gid'];
-               $condition = ['id' => $parent, 'parent' => $parent, 'uid' => $uid];
-               $item = Item::selectFirst($fields, $condition);
-
-               if (DBA::isResult($item)) {
-                       $parent_uri = $item['uri'];
-                       $private    = $item['private'];
-                       $allow_cid  = $item['allow_cid'];
-                       $allow_gid  = $item['allow_gid'];
-                       $deny_cid   = $item['deny_cid'];
-                       $deny_gid   = $item['deny_gid'];
-               }
-       } else {
-               $private = (!empty($_GET['private']) ? intval($_GET['private']) : Item::PUBLIC);
-
-               $allow_cid     = ($private ? '<' . $target['id']. '>' : $a->user['allow_cid']);
-               $allow_gid     = ($private ? '' : $a->user['allow_gid']);
-               $deny_cid      = ($private ? '' : $a->user['deny_cid']);
-               $deny_gid      = ($private ? '' : $a->user['deny_gid']);
-       }
-
-       $poster = $a->contact;
-
-       $uri = Item::newURI($uid);
-
-       $arr = [];
-
-       $arr['guid']          = System::createUUID();
-       $arr['uid']           = $uid;
-       $arr['uri']           = $uri;
-       $arr['parent-uri']    = (!empty($parent_uri) ? $parent_uri : $uri);
-       $arr['wall']          = 1;
-       $arr['contact-id']    = $poster['id'];
-       $arr['owner-name']    = $poster['name'];
-       $arr['owner-link']    = $poster['url'];
-       $arr['owner-avatar']  = $poster['thumb'];
-       $arr['author-name']   = $poster['name'];
-       $arr['author-link']   = $poster['url'];
-       $arr['author-avatar'] = $poster['thumb'];
-       $arr['title']         = '';
-       $arr['allow_cid']     = $allow_cid;
-       $arr['allow_gid']     = $allow_gid;
-       $arr['deny_cid']      = $deny_cid;
-       $arr['deny_gid']      = $deny_gid;
-       $arr['visible']       = 1;
-       $arr['verb']          = $activity;
-       $arr['private']       = $private;
-       $arr['object-type']   = Activity\ObjectType::PERSON;
-
-       $arr['origin']        = 1;
-       $arr['body']          = '[url=' . $poster['url'] . ']' . $poster['name'] . '[/url]' . ' ' . DI::l10n()->t($verbs[$verb][0]) . ' ' . '[url=' . $target['url'] . ']' . $target['name'] . '[/url]';
-
-       $arr['object'] = '<object><type>' . Activity\ObjectType::PERSON . '</type><title>' . $target['name'] . '</title><id>' . $target['url'] . '</id>';
-       $arr['object'] .= '<link>' . XML::escape('<link rel="alternate" type="text/html" href="' . $target['url'] . '" />' . "\n");
-
-       $arr['object'] .= XML::escape('<link rel="photo" type="image/jpeg" href="' . $target['photo'] . '" />' . "\n");
-       $arr['object'] .= '</link></object>' . "\n";
-
-       Item::insert($arr);
-
-       Hook::callAll('post_local_end', $arr);
-
-       return;
-}
-
-function poke_content(App $a)
-{
-       if (!local_user()) {
-               notice(DI::l10n()->t('Permission denied.') . EOL);
-               return;
-       }
-
-       if (empty($_GET['c'])) {
-               return;
-       }
-
-       $contact = DBA::selectFirst('contact', ['id', 'name'], ['id' => $_GET['c'], 'uid' => local_user()]);
-       if (!DBA::isResult($contact)) {
-               return;
-       }
-
-       $name = $contact['name'];
-       $id = $contact['id'];
-
-       $head_tpl = Renderer::getMarkupTemplate('poke_head.tpl');
-       DI::page()['htmlhead'] .= Renderer::replaceMacros($head_tpl,[
-               '$baseurl' => DI::baseUrl()->get(true),
-       ]);
-
-       $parent = (!empty($_GET['parent']) ? intval($_GET['parent']) : '0');
-
-
-       $verbs = DI::l10n()->getPokeVerbs();
-
-       $shortlist = [];
-       foreach ($verbs as $k => $v) {
-               if ($v[1] !== 'NOTRANSLATION') {
-                       $shortlist[] = [$k, $v[1]];
-               }
-       }
-
-       $tpl = Renderer::getMarkupTemplate('poke_content.tpl');
-
-       $o = Renderer::replaceMacros($tpl,[
-               '$title' => DI::l10n()->t('Poke/Prod'),
-               '$desc' => DI::l10n()->t('poke, prod or do other things to somebody'),
-               '$clabel' => DI::l10n()->t('Recipient'),
-               '$choice' => DI::l10n()->t('Choose what you wish to do to recipient'),
-               '$verbs' => $shortlist,
-               '$parent' => $parent,
-               '$prv_desc' => DI::l10n()->t('Make this post private'),
-               '$submit' => DI::l10n()->t('Submit'),
-               '$name' => $name,
-               '$id' => $id
-       ]);
-
-       return $o;
-}
index 2b739278a77d0c418b6cffc69ba595db5780c7c0..2435b54f9c75a17afa36ae53b6856cbc1c3b310c 100644 (file)
@@ -1235,7 +1235,7 @@ class Contact
                }
 
                if (($contact['network'] == Protocol::DFRN) && !$contact['self'] && empty($contact['pending'])) {
-                       $poke_link = DI::baseUrl() . '/poke/?c=' . $contact['id'];
+                       $poke_link = 'contact/' . $contact['id'] . '/poke';
                }
 
                $contact_url = DI::baseUrl() . '/contact/' . $contact['id'];
diff --git a/src/Module/Contact/Poke.php b/src/Module/Contact/Poke.php
new file mode 100644 (file)
index 0000000..a1c2c28
--- /dev/null
@@ -0,0 +1,164 @@
+<?php
+
+namespace Friendica\Module\Contact;
+
+use Friendica\BaseModule;
+use Friendica\Core\Hook;
+use Friendica\Core\Logger;
+use Friendica\Core\Renderer;
+use Friendica\Core\System;
+use Friendica\Database\DBA;
+use Friendica\DI;
+use Friendica\Model;
+use Friendica\Network\HTTPException;
+use Friendica\Protocol\Activity;
+use Friendica\Util\XML;
+
+class Poke extends BaseModule
+{
+       public static function post(array $parameters = [])
+       {
+               if (!local_user() || empty($parameters['id'])) {
+                       return self::postReturn(false);
+               }
+
+               $uid = local_user();
+
+               if (empty($_POST['verb'])) {
+                       return self::postReturn(false);
+               }
+
+               $verb = $_POST['verb'];
+
+               $verbs = DI::l10n()->getPokeVerbs();
+               if (!array_key_exists($verb, $verbs)) {
+                       return self::postReturn(false);
+               }
+
+               $activity = Activity::POKE . '#' . urlencode($verbs[$verb][0]);
+
+               $contact_id = intval($parameters['id']);
+               if (!$contact_id) {
+                       return self::postReturn(false);
+               }
+
+               Logger::info('poke: verb ' . $verb . ' contact ' . $contact_id);
+
+               $contact = DBA::selectFirst('contact', ['id', 'name'], ['id' => $parameters['id'], 'uid' => local_user()]);
+               if (!DBA::isResult($contact)) {
+                       return self::postReturn(false);
+               }
+
+               $a = DI::app();
+
+               $private = (!empty($_GET['private']) ? intval($_GET['private']) : Model\Item::PUBLIC);
+
+               $allow_cid     = ($private ? '<' . $contact['id']. '>' : $a->user['allow_cid']);
+               $allow_gid     = ($private ? '' : $a->user['allow_gid']);
+               $deny_cid      = ($private ? '' : $a->user['deny_cid']);
+               $deny_gid      = ($private ? '' : $a->user['deny_gid']);
+
+               $actor = $a->contact;
+
+               $uri = Model\Item::newURI($uid);
+
+               $arr = [];
+
+               $arr['guid']          = System::createUUID();
+               $arr['uid']           = $uid;
+               $arr['uri']           = $uri;
+               $arr['parent-uri']    = $uri;
+               $arr['wall']          = 1;
+               $arr['contact-id']    = $actor['id'];
+               $arr['owner-name']    = $actor['name'];
+               $arr['owner-link']    = $actor['url'];
+               $arr['owner-avatar']  = $actor['thumb'];
+               $arr['author-name']   = $actor['name'];
+               $arr['author-link']   = $actor['url'];
+               $arr['author-avatar'] = $actor['thumb'];
+               $arr['title']         = '';
+               $arr['allow_cid']     = $allow_cid;
+               $arr['allow_gid']     = $allow_gid;
+               $arr['deny_cid']      = $deny_cid;
+               $arr['deny_gid']      = $deny_gid;
+               $arr['visible']       = 1;
+               $arr['verb']          = $activity;
+               $arr['private']       = $private;
+               $arr['object-type']   = Activity\ObjectType::PERSON;
+
+               $arr['origin']        = 1;
+               $arr['body']          = '[url=' . $actor['url'] . ']' . $actor['name'] . '[/url]' . ' ' . $verbs[$verb][2] . ' ' . '[url=' . $contact['url'] . ']' . $contact['name'] . '[/url]';
+
+               $arr['object'] = '<object><type>' . Activity\ObjectType::PERSON . '</type><title>' . XML::escape($contact['name']) . '</title><id>' . XML::escape($contact['url']) . '</id>';
+               $arr['object'] .= '<link>' . XML::escape('<link rel="alternate" type="text/html" href="' . $contact['url'] . '" />') . "\n";
+
+               $arr['object'] .= XML::escape('<link rel="photo" type="image/jpeg" href="' . $contact['photo'] . '" />') . "\n";
+               $arr['object'] .= '</link></object>' . "\n";
+
+               $result = Model\Item::insert($arr);
+
+               Hook::callAll('post_local_end', $arr);
+
+               return self::postReturn($result);
+       }
+
+       /**
+        * Since post() is called before rawContent(), we need to be able to return a JSON response in post() directly.
+        *
+        * @param bool $success
+        * @return bool
+        */
+       private static function postReturn(bool $success)
+       {
+               if ($success) {
+                       info(DI::l10n()->t('Poke successfully sent.'));
+               } else {
+                       notice(DI::l10n()->t('Error while sending poke, please retry.'));
+               }
+
+               if (DI::mode()->isAjax()) {
+                       System::jsonExit(['success' => $success]);
+               }
+
+               return $success;
+       }
+
+       public static function content(array $parameters = [])
+       {
+               if (!local_user()) {
+                       throw new HTTPException\UnauthorizedException(DI::l10n()->t('You must be logged in to use this module.'));
+               }
+
+               if (empty($parameters['id'])) {
+                       throw new HTTPException\BadRequestException();
+               }
+
+               $contact = DBA::selectFirst('contact', ['id', 'url', 'name'], ['id' => $parameters['id'], 'uid' => local_user()]);
+               if (!DBA::isResult($contact)) {
+                       throw new HTTPException\NotFoundException();
+               }
+
+               Model\Profile::load(DI::app(), '', Model\Contact::getDetailsByURL($contact["url"]));
+
+               $verbs = [];
+               foreach (DI::l10n()->getPokeVerbs() as $verb => $translations) {
+                       if ($translations[1] !== 'NOTRANSLATION') {
+                               $verbs[$verb] = $translations[1];
+                       }
+               }
+
+               $tpl = Renderer::getMarkupTemplate('contact/poke.tpl');
+               $o = Renderer::replaceMacros($tpl,[
+                       '$title'    => DI::l10n()->t('Poke/Prod'),
+                       '$desc'     => DI::l10n()->t('poke, prod or do other things to somebody'),
+                       '$id'       => $contact['id'],
+                       '$verb'     => ['verb', DI::l10n()->t('Choose what you wish to do to recipient'), '', '', $verbs],
+                       '$private'  => ['private', DI::l10n()->t('Make this post private')],
+                       '$loading'  => DI::l10n()->t('Loading...'),
+                       '$submit'   => DI::l10n()->t('Submit'),
+
+               ]);
+
+               return $o;
+       }
+}
index b6ce9bf629fce34bca1d43ec824ce36a6207f6f4..85ec6d8bb49bec61f8930f74cccd4a9af52d5bfb 100644 (file)
@@ -117,6 +117,7 @@ return [
                '/{id:\d+}/conversations' => [Module\Contact::class,           [R::GET]],
                '/{id:\d+}/drop'          => [Module\Contact::class,           [R::GET]],
                '/{id:\d+}/ignore'        => [Module\Contact::class,           [R::GET]],
+               '/{id:\d+}/poke'          => [Module\Contact\Poke::class,      [R::GET, R::POST]],
                '/{id:\d+}/posts'         => [Module\Contact::class,           [R::GET]],
                '/{id:\d+}/update'        => [Module\Contact::class,           [R::GET]],
                '/{id:\d+}/updateprofile' => [Module\Contact::class,           [R::GET]],
index 9805a615436f86c69cfc9727c9c1012fcb796538..cd0253ac665326bdfe0754da0bf2c78d9cae6a8d 100644 (file)
@@ -406,22 +406,6 @@ a {
 .selected-identity img {
   border: 2px solid #ff0000;
 }
-/* poke */
-#poke-desc {
-  margin: 5px 0 10px;
-}
-
-#poke-wrapper  {
-  padding: 10px 0 0px;
-}
-
-#poke-recipient, #poke-action, #poke-privacy-settings {
-  margin: 10px 0 30px;
-}
-
-#poke-recip-label, #poke-action-label, #prvmail-message-label {
-  margin: 10px 0 10px;
-}
 .version-match {
   font-weight: bold;
   color: #00a700;
diff --git a/view/templates/contact/poke.tpl b/view/templates/contact/poke.tpl
new file mode 100644 (file)
index 0000000..88e50cf
--- /dev/null
@@ -0,0 +1,11 @@
+<h2 class="heading">{{$title}}</h2>
+
+<p>{{$desc nofilter}}</p>
+
+<form id="poke-wrapper" action="contact/{{$id}}/poke" method="post">
+       {{include file="field_select.tpl" field=$verb}}
+       {{include file="field_checkbox.tpl" field=$private}}
+       <p class="text-right">
+               <button type="submit" class="btn btn-primary" name="submit" value="{{$submit}}" data-loading-text="{{$loading}}">{{$submit}}</button>
+       </p>
+</form>
diff --git a/view/templates/poke_content.tpl b/view/templates/poke_content.tpl
deleted file mode 100644 (file)
index 91aadcd..0000000
+++ /dev/null
@@ -1,35 +0,0 @@
-
-<h3>{{$title}}</h3>
-
-<div id="poke-desc">{{$desc nofilter}}</div>
-
-
-<div id="poke-wrapper">
-       <form action="poke" method="get">
-
-       <div id="poke-recipient">
-               <div id="poke-recip-label">{{$clabel}}</div>
-               <input id="poke-recip" type="text" size="64" maxlength="255" value="{{$name}}" name="pokename" autocomplete="off" />
-               <input id="poke-recip-complete" type="hidden" value="{{$id}}" name="cid" />
-               <input id="poke-parent" type="hidden" value="{{$parent}}" name="parent" />
-       </div>
-
-       <div id="poke-action">
-               <div id="poke-action-label">{{$choice}}</div>
-               <select name="verb" id="poke-verb-select" >
-               {{foreach $verbs as $v}}
-               <option value="{{$v.0}}">{{$v.1}}</option>
-               {{/foreach}}
-               </select>
-       </div>
-
-       <div id="poke-privacy-settings">
-               <div id="poke-private-desc">{{$prv_desc}}</div>
-               <input type="checkbox" name="private" {{if $parent}}disabled="disabled"{{/if}} value="1" />
-       </div>
-
-       <input type="submit" name="submit" value="{{$submit}}" />
-
-       </form>
-</div>
-
diff --git a/view/templates/poke_head.tpl b/view/templates/poke_head.tpl
deleted file mode 100644 (file)
index cf9ed53..0000000
+++ /dev/null
@@ -1,8 +0,0 @@
-
-<script>
-$(document).ready(function() {
-       $("#poke-recip").name_autocomplete(baseurl + '/search/acl', 'a', true, function(data) {
-               $("#poke-recip-complete").val(data.id);
-       });
-});
-</script>
index 5e7d452043f2f14da9ce2c423a6c11ee760a0487..ac6b5210c68a40e3e0ec53c3acbf64bd5f7b532f 100644 (file)
@@ -2273,7 +2273,7 @@ ul.dropdown-menu li:hover {
  * PAGES
  *********/
 
-.generic-page-wrapper, .videos-content-wrapper,
+.generic-page-wrapper, .contact-content-wrapper, .videos-content-wrapper,
  .suggest-content-wrapper, .common-content-wrapper, .help-content-wrapper,
 .allfriends-content-wrapper, .match-content-wrapper, .dirfind-content-wrapper,
 .delegation-content-wrapper, .notes-content-wrapper,
@@ -2609,14 +2609,6 @@ ul li:hover .contact-wrapper .contact-action-link:hover {
 .photo-album-actions .photos-order-link {
     float: right;
 }
-/* poke */
-#poke-desc {
-    margin: 5px 0 30px;
-}
-#poke-wrapper-end {
-    clear: both;
-}
-
 /* Events page */
 
 .fc .fc-month-view .fc-content .fc-title .item-desc:hover {
index 335b63f16f3211d77bf1f1ab8c8b2ea73b99bf22..b64c7d02026179adcd6b5957a68cf7630fe57f67 100644 (file)
@@ -89,7 +89,7 @@ $(document).ready(function(){
        let $body = $('body');
 
        // show bulk deletion button at network page if checkbox is checked
-       $("body").change("input.item-select", function(){
+       $body.change("input.item-select", function(){
                var checked = false;
 
                // We need to get all checked items, so it would close the delete button
@@ -230,7 +230,7 @@ $(document).ready(function(){
 
        // Dropdown menus with the class "dropdown-head" will display the active tab
        // as button text
-       $("body").on('click', '.dropdown-head .dropdown-menu li a, .dropdown-head .dropdown-menu li button', function(){
+       $body.on('click', '.dropdown-head .dropdown-menu li a, .dropdown-head .dropdown-menu li button', function(){
                toggleDropdownText(this);
        });
 
@@ -264,7 +264,7 @@ $(document).ready(function(){
        // to the input element where the padding value would be at least the width
        // of the button. Otherwise long user input would be invisible because it is
        // behind the button.
-       $("body").on('click', '.form-group-search > input', function() {
+       $body.on('click', '.form-group-search > input', function() {
                // Get the width of the button (if the button isn't available
                // buttonWidth will be null
                var buttonWidth = $(this).next('.form-button-search').outerWidth();
@@ -351,14 +351,14 @@ $(document).ready(function(){
         */
        $("aside")
                .on("shown.bs.offcanvas", function() {
-                       $("body").addClass("aside-out");
+                       $body.addClass("aside-out");
                })
                .on("hidden.bs.offcanvas", function() {
-                       $("body").removeClass("aside-out");
+                       $body.removeClass("aside-out");
                });
 
        // Event listener for 'Show & hide event map' button in the network stream.
-       $("body").on("click", ".event-map-btn", function() {
+       $body.on("click", ".event-map-btn", function() {
                showHideEventMap(this);
        });
 
@@ -400,6 +400,27 @@ $(document).ready(function(){
                .always(function() {
                        $commentSubmit.button('reset');
                });
+       });
+
+       $body.on('submit', '.modal-body #poke-wrapper', function(e) {
+               e.preventDefault();
+
+               let $form = $(this);
+               let $pokeSubmit = $form.find('button[type=submit]').button('loading');
+
+               $.post(
+                       $form.attr('action'),
+                       $form.serialize(),
+                       'json'
+               )
+               .then(function(data) {
+                       if (data.success) {
+                               $('#modal').modal('hide');
+                       }
+               })
+               .always(function() {
+                       $pokeSubmit.button('reset');
+               });
        })
 });
 
index 22afbc9a7cf63394680a9d2f4baaf53ae52e113c..94b29d86035d73d835ccd0d2a07bc4203dc30363 100644 (file)
@@ -117,7 +117,7 @@ function frio_item_photo_links(App $a, &$body_info)
 function frio_item_photo_menu(App $a, &$arr)
 {
        foreach ($arr['menu'] as $k => $v) {
-               if (strpos($v, 'poke?c=') === 0 || strpos($v, 'message/new/') === 0) {
+               if (strpos($v, '/poke') === 0 || strpos($v, 'message/new/') === 0) {
                        $v = 'javascript:addToModal(\'' . $v . '\'); return false;';
                        $arr['menu'][$k] = $v;
                }
@@ -171,7 +171,7 @@ function frio_contact_photo_menu(App $a, &$args)
        // Add to pm and poke links a new key with the value 'modal'.
        // Later we can make conditions in the corresponing templates (e.g.
        // contact_template.tpl)
-       if (strpos($pokelink, 'poke?c=' . $cid) !== false) {
+       if (strpos($pokelink, $cid . '/poke') !== false) {
                $args['menu']['poke'][3] = 'modal';
        }