New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP 7+ requirements, Travis and other small fixes #115

Merged
merged 1 commit into from Jan 18, 2017

Conversation

Projects
None yet
3 participants
@Exileum
Member

Exileum commented Jan 17, 2017

Повышение минимального уровня зависимости до php7 и выше, соответствующие правки связанные с этим (сторонние сервисы и версии php в них). Добавление composer.lock и прочие мелкие исправления, в том числе в шаблонах и т.п.

Начало переноса develop-бранча на отдельную разработку со строгими требованиями к версии php 7+ и всеми современными техниками программирования.

Старый master-бранч остается на кодовой базе версии 2.1.5, соответственно все изменения develop-бранча с этого момента делаются без оглядки на обратную совместимость с ним и с учетом отсутствия необходимости слияния в итоге.

@Exileum Exileum added the Enhancement label Jan 17, 2017

@Exileum Exileum added this to the Версия 2.2.0 milestone Jan 17, 2017

@Exileum Exileum self-assigned this Jan 17, 2017

@Exileum Exileum requested review from VasyOk and leroy0 Jan 17, 2017

<link rel="shortcut icon" href="{SITE_URL}favicon.ico" type="image/x-icon">
<link rel="search" type="application/opensearchdescription+xml" href="{SITE_URL}opensearch_desc.xml" title="{SITENAME} (Forum)" />
<link rel="search" type="application/opensearchdescription+xml" href="{SITE_URL}opensearch_desc_bt.xml" title="{SITENAME} (Tracker)" />
<script type="text/javascript" src="{SITE_URL}styles/js/jquery.pack.js?v={$di->config->get('js_ver')}"></script>
<script type="text/javascript" src="{SITE_URL}styles/js/main.js?v={$di->config->get('js_ver')}"></script>
<script type="text/javascript" src="{SITE_URL}styles/js/jquery.pack.js"></script>

This comment has been minimized.

@Exileum

Exileum Jan 17, 2017

Member

Пока что такое топорное решение, когда руки дойдут до фронта - собираться все равно будет общий файл и там версионность так или иначе тоже не пригодится.

@Exileum

Exileum Jan 17, 2017

Member

Пока что такое топорное решение, когда руки дойдут до фронта - собираться все равно будет общий файл и там версионность так или иначе тоже не пригодится.

@@ -125,36 +125,6 @@
$users_per_day = $total_users;
}
// DB size ... MySQL only

This comment has been minimized.

@Exileum

Exileum Jan 17, 2017

Member

Удаленная топорная ерунда, не несущая никакой полезной нагрузки.

@Exileum

Exileum Jan 17, 2017

Member

Удаленная топорная ерунда, не несущая никакой полезной нагрузки.

@@ -1,7 +1,7 @@
build:
environment:
php:
version: 5.6
version: 7.0.8

This comment has been minimized.

@Exileum

Exileum Jan 17, 2017

Member

Версия выбрана из-за наличия в ней ломающего изменения, вероятно в целом зависимость проекта надо поставить на такой же уровень.

@Exileum

Exileum Jan 17, 2017

Member

Версия выбрана из-за наличия в ней ломающего изменения, вероятно в целом зависимость проекта надо поставить на такой же уровень.

@@ -698,7 +700,8 @@ function delta_time($timestamp_1, $timestamp_2 = TIMENOW, $granularity = 'auto')
*/
function get_select($select, $selected = null, $return_as = 'html', $first_opt = '&raquo;&raquo; Выбрать ')
{
$select_ary = array();
$select_name = '';

This comment has been minimized.

@leroy0

leroy0 Jan 18, 2017

Collaborator

Скорее надо в switch добавить
default: throw new Exception('incorrect select'); или что-то типа того

@leroy0

leroy0 Jan 18, 2017

Collaborator

Скорее надо в switch добавить
default: throw new Exception('incorrect select'); или что-то типа того

This comment has been minimized.

@leroy0

leroy0 Jan 18, 2017

Collaborator

Хотя пофиг на самом деле.

@leroy0

leroy0 Jan 18, 2017

Collaborator

Хотя пофиг на самом деле.

@leroy0

leroy0 approved these changes Jan 18, 2017

Правки убраны, не актуально

@Exileum Exileum merged commit f4a70a1 into torrentpier:develop Jan 18, 2017

4 checks passed

Scrutinizer 10 new issues
Details
SensioLabsInsight Code quality OK.
Details
continuous-integration/styleci/pr The StyleCI analysis has passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Exileum Exileum deleted the Exileum:php7_requirement branch Jan 30, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment