Closed (fixed)
Project:
VK CrossPoster
Version:
7.x-2.0
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Issue tags:
Reporter:
Created:
7 Jul 2014 at 06:32 UTC
Updated:
18 Jan 2016 at 14:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
anpolimusComment #2
anpolimusHi.
Please, review my patch.
Comment #3
anpolimusUP. Please, somebody review m patch.
Comment #4
kaztur commented@anpolimus, great Thank You for the patch!
Reposting tags works fine:

But there is an error in dblog:
Deprecated function: curl_setopt(): The usage of the @filename API for file uploading is deprecated. Please use the CURLFile class instead в функции vkxp_query() (строка 286 в файле /sites/all/modules/vkxp/vkxp.module).And a little feature request - is there just change any line in patch to add only in public search by hastags? It is realizing by adding @publicname to each hashtag (e.g. "#nyhashtag@mypublic)".
(Documentation in Vk: https://vk.com/pages?oid=-2158488&p=%D0%9B%D0%BE%D0%BA%D0%B0%D0%BB%D1%8C...)
Comment #5
kaztur commentedAlso, I suppose the line
$hash_tags[] = '#' . str_replace(' ', '_', $item['name']);would be better to change to
$hash_tags[] = '#' . str_replace(' ', '', $item['name']);After all, fused writing of hashtags containing two or more words in Vk as to me is more often than via underscore.
Comment #6
kaztur commentedSome problem with compatability with this patch: https://www.drupal.org/node/1677850
When posting to vk with rules and scheduler, hashtags are empty. Is there any solution?
Comment #7
anpolimusThanks for your answer.
I'll retest it again soon.
Comment #8
anpolimusHi, I've retested my code with action and everything is working properly.
http://screencloud.net/v/jvNL
Important thing - tags field should be configured for hash tags support
http://screencloud.net/v/7Sc1
Comment #9
anpolimusComment #10
anpolimusA new patch with
has added.
About publicname. Is it sufficient to get this data from node field (like it is done for hash tag field)?
@kaztur, please answer.
Comment #11
anpolimusNeeds review again :)
Comment #12
anpolimusUp
Comment #13
kaztur commented@anpolimus, #6 repeats again.
In dblog I have to logs, but they seems to be not issues to hashtags:
So the first notice (undefined index) appears only id we schedule node publish (no notice when publishing immediately by saving node).
Comment #14
anpolimusComment #15
anpolimusComment #16
anpolimus@kaztur, please retest with the latest patch.
Comment #17
kaztur commented@anpolimus, great thanks!
I have tested the last patch and don't know while, which kind bug is it, but instead of the last hashtag there was posted penultimate tag twice.
E.g.: if we have 3 tags for node: football, ball, play; we will have hashtags posted #football, #ball, #ball.
The same behavior if we post node directly by saving node (without scheduler and rules).
Tested it twice with 2 different nodes.
Comment #18
anpolimusThanks for testing.
Investigating that.
Comment #19
anpolimusA new patch for testing is ready.
Comment #20
kaztur commented@anpolimus, now it works fine. Finally great thank You!
If every maintainer would have been like you... :)
Comment #21
kaztur commentedComment #22
anpolimusComment #24
anpolimusLatest patch has been commited. to dev version.
Comment #25
anpolimusTask marked as fixed. This commit would be at 7.x-2.21 version.
Comment #26
anpolimus