modules.vk

Complexity

Total Complexity

12

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Importance

Changes

0

4 Functions

Rating	Name	Duplication	Size	Complexity
A	check_post()	0	12	1
B	dlpic()	0	22	7
A	sonyan_post()	0	11	2
A	module_init()	0	10	2

#!/usr/bin/python
# -*- coding: utf-8 -*-
from telegram.ext import CommandHandler
from modules.logging import vk, vk_add
from telegram import InputMediaPhoto
import requests
import time

owner = "-98881019"
count = "1"
offset = 0


def module_init(gd):
    global channel, token, database
    channel = gd.config["channel"]
    token = gd.config["vk_token"]
    commands = gd.config["commands"]
    database = gd.config["database"]
    for command in commands:
        gd.dp.add_handler(CommandHandler(command, sonyan_post))
    jobQueue = gd.updater.job_queue
    jobQueue.run_repeating(callback=sonyan_post, interval=60, first=5, context="@"+channel, name='RepeatingJob')


def sonyan_post(bot, update):
    unixtime, post_id = check_post(owner, offset, count, token)

The variable token does not seem to be defined.

    current_time, unixtime_old = vk(bot, update)
    if unixtime > unixtime_old:
        print("New post!")
        images = dlpic(owner, offset, count, token)
    else:
        return
    bot.send_media_group(chat_id="@"+channel, media=images)

The variable channel does not seem to be defined.

    vk_add(bot, update, current_time, unixtime, post_id)
    time.sleep(10)


def check_post(owner, offset, count, token):
    wallposts = requests.get("https://api.vk.com/method/wall.get?"+
                            "owner_id="+owner+
                             "&offset="+str(offset)+
                             "&count="+count+
                             "&filter="+owner+
                             "&access_token="+token+
                             "&v=5.60")
    serverjson = wallposts.json()
    date = serverjson["response"]["items"][0]["date"]
    post_id = serverjson["response"]["items"][0]["id"]
    return date, post_id


def dlpic(owner, offset, count, token):
    wallposts = requests.get("https://api.vk.com/method/wall.get?"+
                             "owner_id="+owner+
                             "&offset="+str(offset)+
                             "&count="+count+
                             "&filter="+owner+
                             "&access_token="+token+
                             "&v=5.60")
    serverjson = wallposts.json()
    try:
        attachments = serverjson["response"]["items"][0]["attachments"]
    except:
        return None, None
    media_list = []
    for i in attachments:
        if "photo" not in i:
            continue
        photo = i["photo"]
        link = photo[max((x for x in photo if x.startswith("photo_")), key=lambda x: int(x.split("_")[1]))]

The variable x does not seem to be defined in case the for loop on line 67 is not entered. Are you sure this can never be the case?

        media = InputMediaPhoto(media=link)
        media_list.append(media)
    return media_list