master #13

Merged

asmeron merged 52 commits from Igor1/ublexec:master into develop

Igor1 commented

Beta версия.

Igor1 added 27 commits

881f9f08ec Частично код приведён стандарту Стэнфордского университет

0c3dfb06e2 Merge branch 'devel'

b16a9777b9 Исправлен Plug

c9c1d17697 Merge branch 'devel'

dba10809de Исправлена кодировка

80da50c4ab Merge branch 'devel'

2ff7301390 Fix

fcb9f58945 Merge branch 'devel'

af6e0d204a Fix css

6d5f7c9759 Merge branch 'devel'

b771944d56 fix

85022bbc33 fix

605033cde0 fix

48edcdf6c4 dsdvdvdMerge branch 'master' into devel

0d28917d4f Исправлено формирование команды

373089f949 Добавлена проверка группы wheel

699be1fcca Исправлен проверка группы wheel

e17da88c44 Исправлен frame

a9933b7f99 Исправил загрузку картинок

847b9f85a3 Исправлен About

4e1af25703 Исправлен дизайн кнопок

cecdb1e3aa Исправлен скриншот

aaea04c259 Исправлена логика активности элемента Приоритет

85b6407174 Исправлено окно о программе

48c1f04acc Исправлен дизан

e07d5e13ef Добавлен .pot файл

6886ce9320 Добавлен .pot файл

Igor1 added 1 commit

0813b57ef7 Исправлена локализация

Igor1 added 1 commit

4f138eee7d Исправлен баг с wheel

Igor1 added 2 commits

9bd77a4e61 Исправлен дизайн

06153fad01 Исправлен скриншот

Igor1 added 1 commit

d4ef7ed529 Исправлен файл .desktop

asmeron requested changes

asmeron left a comment

Owner

Очень много глупых мелких ошибок !

source/ublexec.cc Outdated

													
					            break;

					        }

					        if (entry->pw_uid >= 1000 && entry->pw_uid!=65534){

asmeron commented

Owner

Пользователи неправильно список формируется, вот для понимания, пользователи:

#!/usr/bin/env bash
UID_MIN=$(grep "^UID_MIN" /etc/login.defs | xargs | cut -d ' ' -f 2)
UID_MAX=$(grep "^UID_MAX" /etc/login.defs | xargs | cut -d ' ' -f 2)
while read USER; do
    UID_USER=$(cut -d: -f2 <<< ${USER})
    if (( ${UID_USER} >= ${UID_MIN} && ${UID_USER} <= ${UID_MAX} )) || (( ${UID_USER} == 0 )); then
        echo "${USER}" | cut -d: -f1
    fi
done < <(getent passwd | cut -d: -f1,3)

Igor1 commented

Poster

Исправил.
b65ce014f3
f78e2a0533

asmeron commented

Owner

Ничего не исправил, если не понял, что делать, спроси в группе !

source/ublexec.cc Outdated

													
					        spinPriority->set_increments(1.0, -1.0);

					        scalePriority->set_range(-20, 19);

					        scalePriority->set_value(0);

					        lblTimeEpriorityLow->set_text("19 (Низкий)");

asmeron commented

Owner

Почему текст без перевода а напрямую на русском ???

Igor1 commented

Poster

Исправил.
e730c17e48

source/ublexec.cc Outdated

													
					        scalePriority->set_range(-20, 19);

					        scalePriority->set_value(0);

					        lblTimeEpriorityLow->set_text("19 (Низкий)");

					        lblTime4EpriorityHigh->set_text("-20 (Высокий)");

asmeron commented

Owner

Почему текст без перевода а напрямую на русском ???

Igor1 commented

Poster

Исправил.
e730c17e48

source/ublexec.cc Outdated

													
					        spinPriority->set_increments(1.0, -1.0);

					        scalePriority->set_range(-20, 19);

					        scalePriority->set_value(0);

					        lblTimeEpriorityLow->set_text("19 (Низкий)");

asmeron commented

Owner

Почему текст без перевода а напрямую на русском ???

И далее исправить везде это недоразумение !!

Igor1 commented

Poster

Исправил.
e730c17e48

source/ublexec.cc Outdated

													
					void MainWindow::synopsis_show(){

					void MainWindow::synopsis_show() {

					    system("xdg-open https://wiki.ublinux.ru/ru/home &");

asmeron commented

Owner

Открывать в зависимости от локализации !!

Igor1 commented

Poster

Исправил.
e730c17e48

source/ublexec.cc Outdated

													
					    if (path_file.length()>0){

					        cmd = "nohup " + str_nice_cmd + str_cmd_terminal + str_variants_root + " xdg-open '" + path_file + "' ";

					    if (path_file.length() > 0) {

					        cmd = str_nice_cmd + str_cmd_terminal + str_variants_root + " xdg-open '" + path_file + "' ";

asmeron commented

Owner

Скорректировать ТЗ по командам, т.к. в ТЗ не вижу xdg-open

source/ublexec.cc Outdated

													
					        cmd = str_nice_cmd + str_cmd_terminal + str_variants_root + " " + user_cmd;

					    }

					    if (cmd.find("-e su ")!=string::npos){

					    if ((cmd.find("-e su ") != string::npos) || (cmd.find("-x su ") != string::npos)) {

asmeron commented

Owner

Скорректировать ТЗ по командам, т.к. в ТЗ не вижу -x su

Igor1 commented

Poster

Это часть команды и это проверка присутствия в строке , а не выполнение !!!
Вот например.
nohup xfce4-terminal -x su -l superadmin -c " DISPLAY=$DISPLAY gimp " > /dev/null 2>&1

asmeron commented

Owner

Если это часть команды, то должна быть в ТЗ !

Igor1 commented

Poster

Это часть от xfce4-terminal -x запуск терминала, su в представление не нуждается.Это две разных команды, не одна . Я проверяю, есть ли в строке подстрока, а не не выполняю команду.

asmeron commented

Owner

Сделать, что написал выше !

ublexec.desktop Outdated

													
					Categories=XFCE;GTK;Settings;DesktopSettings;X-XFCE-SettingsDialog;X-XFCE-SystemSettings;X-UBL-SettingsManager;X-UBL-SystemSettings;

					X-UBLPluggable=true

					Categories=XFCE;GTK;Settings;DesktopSettings;X-XFCE-SettingsDialog;X-XFCE-SystemSettings;;X-UBL-SettingsManager;X-UBL-SystemSettings;

asmeron commented

Owner

Писал об этом, это утилита не настроек, каждый раз одни и те-же ошибки !

Igor1 commented

Poster

Исправил.
b0a7508b38

ublexec.pot

													
					msgid "https://wiki.ublinux.com"

					msgstr ""

					msgid "About the program"

asmeron commented

Owner

Это говорил убрать !!
Получается файл ublexec_ru.pot не соответствует ublexec_ru.po
Проверить вручную каждую строку !!

Igor1 commented

Poster

Исправил.
e71e07ae31

Igor1 added 1 commit

e730c17e48 Исправлена локализация

Igor1 added 1 commit

e71e07ae31 Исправлен .pot файл

Igor1 added 1 commit

b0a7508b38 Исправлен .desktop файл

Igor1 added 1 commit

d14d1f2ede Исправлен about

Igor1 added 1 commit

b65ce014f3 Исправлена логика поиска пользователей

Igor1 added 1 commit

f78e2a0533 Исправлена активация пользователя

Igor1 added 1 commit

485b50e634 Чистка кода

Igor1 added 1 commit

07fc433844 Чистка кода

Igor1 added 1 commit

d786190dca Исправлен .desktop файл

Igor1 added 1 commit

e613b3445a Исправлен desktop файл

Igor1 added 1 commit

db4bb08465 Исправлен desktop файл

Igor1 added 1 commit

f086f5daec Исправлены иконки

Igor1 added 1 commit

709176c729 Fix

Igor1 added 1 commit

cb31253727 Убрал флаг отладки

Igor1 added 1 commit

10eb497496 Исправлена логика открытия файлов

Igor1 added 1 commit

9dbbe7b5e1 Исправлена локализация "User Name"

Igor1 added 1 commit

bab8e36a59 Исправлен дизайн

Igor1 added 1 commit

905a1e0939 Исправлен дизайн

Igor1 added 1 commit

5410438b6a Исправлен дизайн

Igor1 added 1 commit

88fb010983 fix

asmeron requested changes

asmeron left a comment

Owner

Исправить !

Makefile

													
						@install -Dm644 -t /usr/share/${PKGNAME}/css/ style.css 

						@install -Dm644 -t /usr/share/${PKGNAME}/images/ bg_top.png

						@install -Dm644 -t /usr/share/${PKGNAME}/images/ ${PKGNAME}.svg

						@install -Dm644 -t /usr/share/${PKGNAME}/images/ "ru.ublinux.${PKGNAME}.svg"

asmeron commented

Owner

Уже писал об этом !
Дубликат который храниться в двух местах:

@install -Dm644 -t /usr/share/icons/hicolor/scalable/apps/ "ru.ublinux.${PKGNAME}.svg"
@install -Dm644 -t /usr/share/${PKGNAME}/images/ "ru.ublinux.${PKGNAME}.svg"

Для чего ??

source/ublexec.cc

													
					        str_cmd_terminal = "xterm -e ";

					    if (chbTerminal->get_active() == true) {

					        string name_terminal =  cmbTerminal->get_active_text();

					        if (name_terminal.length() != 0){

asmeron commented

Owner

Перечень возможных терминалов вынести в массив, который описать в заголовке !
xfce4-terminal -x
konsole -e
xterm -e

source/ublexec.cc

													
					    string file_name = "";

					    for (const auto & entry: fs::directory_iterator(path)) {

					        file_name = entry.path().filename().string();

					        if (file_name == "konsole") {

asmeron commented

Owner

Использовать массив терминалов для выборки и обработки

source/ublexec.cc

													
					    //    int uid_max = stoi(str_uid_max);

					    //}

					    //else{

					    int uid_max = 65534;

asmeron commented

Owner

Задача по этому поводу существует, исправить !

asmeron referenced this pull request

Вывод только пользователей в рамках UID_MIN - UID_MAX + root #14

asmeron referenced this pull request

Возможные варианты эмуляторов термина вынести в массив, с зависящими входящими аргументами #15

asmeron referenced this pull request

Удалить дубликаты иконок #16

asmeron merged commit 987a5e8d53 into develop 3 years ago

asmeron referenced this issue from a commit

Merge pull request 'master' (#13) from Igor1/ublexec:master into develop

Reviewers

asmeron requested changes

The pull request has been merged as 987a5e8d53.

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Applications/ublexec#13