Skip to content
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

Installer fix #3991

Merged
merged 113 commits into from
Dec 14, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
113 commits
Select commit Hold shift + click to select a range
20705fd
initial commits
craigh Nov 24, 2019
77b1d9f
temporarily disable jquery.mmenu because causing errors on composer u…
craigh Nov 24, 2019
b42a215
force composer to use my branch of symfony requirements-checker
craigh Nov 24, 2019
0440bd8
disable Liip/Imagine because causes problem on Kernel instantiation (…
craigh Nov 24, 2019
89bcd72
finish disable of jQuery.mmenu
craigh Nov 25, 2019
9c77a59
correct argument casting because expression language is used in the d…
craigh Nov 25, 2019
d71f621
update composer.lock
craigh Nov 25, 2019
8d1e5ab
more disabling of liip/imaginebundle
craigh Nov 25, 2019
61d5769
update composer.lock
craigh Nov 25, 2019
43d2fee
Revert early removal of mmenu and correctly reinstall. fixes #3982
craigh Nov 25, 2019
c6ccfc9
correct regression refs 64ea5ccdf364427930960960bb77f62ba2d5a8bb and …
craigh Nov 25, 2019
1f344b2
change !tagged to !tagged_iterator as former was deprecated in s4.4
craigh Nov 25, 2019
3ef830f
allow custom tags (e.g/ !tagged_iterator) when linting
craigh Nov 25, 2019
945378c
also lint container in build
craigh Nov 25, 2019
0464b55
use local console
craigh Nov 25, 2019
ffe3bf1
corrections for @Guite
craigh Nov 25, 2019
5e02851
styleci fixes
craigh Nov 25, 2019
ca4250e
disable unit tests :-(
craigh Nov 25, 2019
b6f2af7
ignore other vendor dir if created for some reason
craigh Nov 25, 2019
559808a
fix js translation extractor
craigh Nov 25, 2019
932f563
temp disable because `session.storage` is private
craigh Nov 25, 2019
ebd5a0a
ini_set requires string arg
craigh Nov 25, 2019
98ef2a1
correct service definitions
craigh Nov 25, 2019
84b66f0
several fixes for #3991
Guite Nov 26, 2019
b6b4de9
cs
Guite Nov 26, 2019
a422073
quote reserved words in sql
craigh Nov 27, 2019
b94ef16
add more debugger to travis
craigh Nov 27, 2019
995d208
don't instantiate router if not needed.
craigh Nov 27, 2019
3399a5a
create interface for ThemeEntityRepository class and configure like o…
craigh Nov 27, 2019
f2dabfc
ci
craigh Nov 27, 2019
dfd3933
set some properties in child class to avoid unneeded container requests
craigh Nov 27, 2019
a8a16d9
Entity expecting integers not bool
craigh Nov 27, 2019
de8ab43
avoid additional theme regeneration making theme bundles inactive
Guite Nov 30, 2019
3f9f271
begin
craigh Nov 30, 2019
c57fe17
Merge incomplete branch 'refactor' into installer branch
craigh Nov 30, 2019
f05c1ce
begin refactoring
craigh Dec 1, 2019
c0edea2
Apply fixes from StyleCI
craigh Dec 1, 2019
f2ae243
Merge pull request #3994 from zikula/analysis-rd4l1g
craigh Dec 1, 2019
328bc2b
update config path
craigh Dec 1, 2019
7ab295d
prefix `pdo_` to db drivers
craigh Dec 2, 2019
bf01e90
Revert "prefix `pdo_` to db drivers"
craigh Dec 2, 2019
8c170ba
add `pdo_` prefix on initialization
craigh Dec 2, 2019
728de16
set base_url on test install
craigh Dec 2, 2019
6186ac8
add forgotten constructor
craigh Dec 2, 2019
7d06f82
await filewrite
craigh Dec 3, 2019
54f5991
Revert "await filewrite"
craigh Dec 3, 2019
9aafbb1
update vendors.
craigh Dec 3, 2019
52dc30c
ci
craigh Dec 3, 2019
e9e21ba
switch order when protecting files
craigh Dec 4, 2019
986971e
disable upgrade tests for now
craigh Dec 4, 2019
baebc22
add properties used in child classes
craigh Dec 8, 2019
f134037
correct class name in comment
craigh Dec 8, 2019
bc2ff60
correct issue obtaining parameters file
craigh Dec 8, 2019
7c5b4e3
add twig/twig as explicit dependency even though it was being include…
craigh Dec 8, 2019
9f3a2ac
remove templating key from framework config - it is not supported in …
craigh Dec 8, 2019
41dd74b
correct service definition for EventEnabledTwigEngine
craigh Dec 8, 2019
10439db
correct service definition for ControllerResolver (not sure this is t…
craigh Dec 8, 2019
85fa9f2
add import for global class
craigh Dec 8, 2019
1f194cc
Convert standard EventDispatcher to LegacyEventDispatcherProxy to all…
craigh Dec 8, 2019
f8f6443
vendor update
craigh Dec 8, 2019
2078cd7
use local var
craigh Dec 8, 2019
d2f0b07
use local var
craigh Dec 8, 2019
12d665d
move constants to interface
craigh Dec 8, 2019
3903028
use proper namespaced twig locations
craigh Dec 8, 2019
3ab4b19
refactor ajax upgrader to new services
craigh Dec 8, 2019
055a34f
rename Manager to Helper
craigh Dec 8, 2019
57b9971
pretty sure we want to remove bundle state
craigh Dec 9, 2019
f6183ed
standard director is `log` not `logs` and the directory is autocreate…
craigh Dec 9, 2019
777bf3a
more removing of bundle state
craigh Dec 9, 2019
c76988a
better directory file scanning
craigh Dec 9, 2019
e88168c
add specific arg to service declaration
craigh Dec 9, 2019
4632a6e
using twig namespacing
craigh Dec 9, 2019
d68c61b
Apply fixes from StyleCI
craigh Dec 9, 2019
9aa51ff
Merge pull request #3995 from zikula/analysis-lKVOeO
craigh Dec 9, 2019
cd0b5e5
correct typehint to interface @Guite this probably needs updating in …
craigh Dec 9, 2019
b6aca47
removed obsolete argument
Guite Dec 10, 2019
06d7eba
added changelog entry for vendor update
Guite Dec 10, 2019
b1d6a7b
use twig namespacing
Guite Dec 10, 2019
8e05905
use loader class from Twig instead of Symfony Twig bundle
Guite Dec 10, 2019
ec5b761
regenerated Routes
Guite Dec 10, 2019
435ba16
bring back custom admin icon from 2.0
Guite Dec 10, 2019
2b7468e
refs 777bf3a91cb5662c4205ef246111a1b91dd393a0
craigh Dec 10, 2019
ce46ff9
errors file moved locations
craigh Dec 10, 2019
ef6288c
ci correct case
craigh Dec 10, 2019
57b0f22
remove duplicated function
craigh Dec 11, 2019
614c8ec
removed unneeded code
Guite Dec 11, 2019
11f8859
regenerated Routes
Guite Dec 12, 2019
8014c22
update service definition for EventEnabledTwigEngine (this is not wor…
craigh Dec 12, 2019
7dc8e8a
correct definition for zikula's child class of Twig\Environment
craigh Dec 13, 2019
00db42a
use interface class name instead of self::
craigh Dec 13, 2019
37ed8c7
use null for arg instead of empty array
craigh Dec 13, 2019
804177b
twig namespacing
craigh Dec 13, 2019
e8fcfda
Revert "use null for arg instead of empty array"
craigh Dec 13, 2019
246b28a
correct Interface Constant usage
craigh Dec 13, 2019
09d4c6c
use setter injection for Logger because it is a private service. @Gui…
craigh Dec 13, 2019
aa4217c
revert move of comment since styleCI hates it @Guite
craigh Dec 13, 2019
080b366
Move global definitions to service definition directly
craigh Dec 14, 2019
573a84a
start rough guide for refactoring
craigh Dec 14, 2019
dc746f7
twig namesapcing
craigh Dec 14, 2019
0c55053
fully remove html5shiv. refs #2416
craigh Dec 14, 2019
4cf4ce0
cleanup uneeded code
craigh Dec 14, 2019
838fa1c
revert changes in asset path locations.
craigh Dec 14, 2019
8e23be0
add helpful comment
craigh Dec 14, 2019
2954015
add minor feedback to user waiting for page to load
craigh Dec 14, 2019
9ad4fcd
correct reversion
craigh Dec 14, 2019
1bc48e2
correct swiftmailer usage
craigh Dec 14, 2019
2f8d7c0
twig namespacing
craigh Dec 14, 2019
d7130ed
stages are not services and must implement (for the moment) container…
craigh Dec 14, 2019
cd54c3c
javascript expects actual boolean values
craigh Dec 14, 2019
725ad16
correct injection of translator in Command
craigh Dec 14, 2019
3b1a0db
Apply fixes from StyleCI
craigh Dec 14, 2019
28be71b
Merge pull request #4001 from zikula/analysis-VrrGBV
craigh Dec 14, 2019
6366fc2
re-enable liip/imagine
craigh Dec 14, 2019
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
33 changes: 33 additions & 0 deletions .env
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
# In all environments, the following files are loaded if they exist,
# the latter taking precedence over the former:
#
# * .env contains default values for the environment variables needed by the app
# * .env.local uncommitted file with local overrides
# * .env.$APP_ENV committed environment-specific defaults
# * .env.$APP_ENV.local uncommitted environment-specific overrides
#
# Real environment variables win over .env files.
#
# DO NOT DEFINE PRODUCTION SECRETS IN THIS FILE NOR IN ANY OTHER COMMITTED FILES.
#
# Run "composer dump-env prod" to compile .env files for production use (requires symfony/flex >=1.2).
# https://symfony.com/doc/current/best_practices/configuration.html#infrastructure-related-configuration

###> symfony/framework-bundle ###
APP_ENV=dev
APP_SECRET=f936f24ee90871ba5c4904768f2b1950
#TRUSTED_PROXIES=127.0.0.0/8,10.0.0.0/8,172.16.0.0/12,192.168.0.0/16
#TRUSTED_HOSTS='^localhost|example\.com$'
###< symfony/framework-bundle ###

###> symfony/mailer ###
# MAILER_DSN=smtp://localhost
###< symfony/mailer ###

###> doctrine/doctrine-bundle ###
# Format described at https://www.doctrine-project.org/projects/doctrine-dbal/en/latest/reference/configuration.html#connecting-using-a-url
# For an SQLite database, use: "sqlite:///%kernel.project_dir%/var/data.db"
# For a PostgreSQL database, use: "postgresql://db_user:[email protected]:5432/db_name?serverVersion=11"
# IMPORTANT: You MUST configure your db driver and server version, either here or in config/packages/doctrine.yaml
DATABASE_URL=mysql://db_user:[email protected]:3306/db_name?serverVersion=5.7
###< doctrine/doctrine-bundle ###
4 changes: 4 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
/composer
/composer.phar
/bin/
/src/bin/*
!/src/bin/console
/src/app/config/custom_*.yml
/src/app/config/dynamic/*
!/src/app/config/dynamic/default.yml
Expand All @@ -12,10 +14,12 @@
!/src/themes/RssTheme
/src/app/Resources/translations/*
/src/vendor/
/vendor/
/src/web/*
!/src/web/uploads/
/.vagrant


!/test_dbs/*.sql
/src/lib/Zikula/Bundle/CoreBundle/Tests/Functional/cache/

Expand Down
54 changes: 26 additions & 28 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,17 +29,15 @@ before_install:
- composer run-script post-install-cmd

- cd src
- php bin/console lint:container
- php bin/console lint:yaml app/config/
- php bin/console lint:yaml system/
- php bin/console lint:yaml system/ --parse-tags
- php bin/console lint:yaml themes/
- php bin/console lint:yaml lib/Zikula/
- php bin/console lint:yaml lib/Zikula/ --parse-tags
- php bin/console lint:twig system/ lib/Zikula/
# twig linting in themes won't work because the themes are not active bundles

- cd ..
- ./bin/simple-phpunit

- cd src
# - php bin/simple-phpunit

# extract en translations
- php -dmemory_limit=2G bin/console translation:extract en --output-format=po --output-dir=app/Resources/translations --enable-extractor=jms_i18n_routing --dir=system --dir=lib/Zikula/Bundle
Expand All @@ -48,46 +46,46 @@ install:
# create test db
- mysql -e 'create database zk_test;'
# run the installer
- php bin/console zikula:install:start -n --database_user=root --database_name=zk_test --password=12345678 [email protected] --router:request_context:host=localhost
- php bin/console zikula:install:finish
- php bin/console zikula:install:start -n --database_user=root --database_name=zk_test --password=12345678 [email protected] --router:request_context:host=localhost --router:request_context:base_url='/' -vvv
- php bin/console zikula:install:finish -vvv
# drop the test db
- mysql -e 'drop database zk_test'

# create an upgrade db for 143
- mysql -e 'create database zk_test;'
- mysql zk_test < ../test_dbs/core143.sql
# - mysql -e 'create database zk_test;'
# - mysql zk_test < ../test_dbs/core143.sql
# run the upgrade from the 143 db
- 'sed -i -E "s/core_installed_version:(.*)/core_installed_version: ''1.4.3''/" app/config/custom_parameters.yml'
- php bin/console zikula:upgrade -n --username=admin --password=12345678 --router:request_context:host=localhost --router:request_context:scheme=http --router:request_context:base_url='/'
# - 'sed -i -E "s/core_installed_version:(.*)/core_installed_version: ''1.4.3''/" app/config/custom_parameters.yml'
# - php bin/console zikula:upgrade -n --username=admin --password=12345678 --router:request_context:host=localhost --router:request_context:scheme=http --router:request_context:base_url='/'
# drop the test db
- mysql -e 'drop database zk_test'
# - mysql -e 'drop database zk_test'

# create an upgrade db for 144
- mysql -e 'create database zk_test;'
- mysql zk_test < ../test_dbs/core144.sql
# - mysql -e 'create database zk_test;'
# - mysql zk_test < ../test_dbs/core144.sql
# run the upgrade from the 144 db
- 'sed -i -E "s/core_installed_version:(.*)/core_installed_version: ''1.4.4''/" app/config/custom_parameters.yml'
- php bin/console zikula:upgrade -n --username=admin --password=12345678 --router:request_context:host=localhost --router:request_context:scheme=http --router:request_context:base_url='/'
# - 'sed -i -E "s/core_installed_version:(.*)/core_installed_version: ''1.4.4''/" app/config/custom_parameters.yml'
# - php bin/console zikula:upgrade -n --username=admin --password=12345678 --router:request_context:host=localhost --router:request_context:scheme=http --router:request_context:base_url='/'
# drop the test db
- mysql -e 'drop database zk_test'
# - mysql -e 'drop database zk_test'

# create an upgrade db for 145
- mysql -e 'create database zk_test;'
- mysql zk_test < ../test_dbs/core145.sql
# - mysql -e 'create database zk_test;'
# - mysql zk_test < ../test_dbs/core145.sql
# run the upgrade from the 145 db
- 'sed -i -E "s/core_installed_version:(.*)/core_installed_version: ''1.4.5''/" app/config/custom_parameters.yml'
- php bin/console zikula:upgrade -n --username=admin --password=12345678 --router:request_context:host=localhost --router:request_context:scheme=http --router:request_context:base_url='/'
# - 'sed -i -E "s/core_installed_version:(.*)/core_installed_version: ''1.4.5''/" app/config/custom_parameters.yml'
# - php bin/console zikula:upgrade -n --username=admin --password=12345678 --router:request_context:host=localhost --router:request_context:scheme=http --router:request_context:base_url='/'
# drop the test db
- mysql -e 'drop database zk_test'
# - mysql -e 'drop database zk_test'

# create an upgrade db for 146
- mysql -e 'create database zk_test;'
- mysql zk_test < ../test_dbs/core146.sql
# - mysql -e 'create database zk_test;'
# - mysql zk_test < ../test_dbs/core146.sql
# run the upgrade from the 146 db
- 'sed -i -E "s/core_installed_version:(.*)/core_installed_version: ''1.4.6''/" app/config/custom_parameters.yml'
- php bin/console zikula:upgrade -n --username=admin --password=12345678 --router:request_context:host=localhost --router:request_context:scheme=http --router:request_context:base_url='/'
# - 'sed -i -E "s/core_installed_version:(.*)/core_installed_version: ''1.4.6''/" app/config/custom_parameters.yml'
# - php bin/console zikula:upgrade -n --username=admin --password=12345678 --router:request_context:host=localhost --router:request_context:scheme=http --router:request_context:base_url='/'
# drop the test db
- mysql -e 'drop database zk_test'
# - mysql -e 'drop database zk_test'

script:
- cd ..
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG-3.0.md
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,7 @@ CHANGELOG - ZIKULA 3.0.x
- doctrine/reflection installed in 1.0.0
- egulias/email-validator installed in 2.1.11
- friendsofsymfony/jsrouting-bundle updated from 1.6.3 to 2.4.0
- guzzlehttp/guzzle updated from 6.4.1 to 6.5.0
- imagine/imagine updated from 0.7.1 to 1.2.2
- jms/i18n-routing-bundle updated from 2.0.0 to 3.0.3
- jms/translation-bundle updated from 1.3.2 to 1.4.4
Expand Down
4 changes: 4 additions & 0 deletions Refactoring for 3.0.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@

- Twig
- change all template names from e.g. Bundle:Controller:Action.html.twig to @Bundle/Controller/Action.html.twig
- Modules and themes retain the `Module` or `Theme` suffix but Bundles do not.
8 changes: 4 additions & 4 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,9 @@
"zikula-scripts": [
"Zikula\\Composer\\CopyRequirementsFile::copy",
"composer dump-autoload",
"Sensio\\Bundle\\DistributionBundle\\Composer\\ScriptHandler::buildBootstrap",
"Sensio\\Bundle\\DistributionBundle\\Composer\\ScriptHandler::clearCache",
"Sensio\\Bundle\\DistributionBundle\\Composer\\ScriptHandler::installAssets",
"src/bin/console cache:clear --env prod",
"src/bin/console cache:clear",
"src/bin/console assets:install --relative src/web",
"Zikula\\Composer\\ManuallyInstallAssets::install",
"Zikula\\Composer\\LessGenerator::generateCombinedBootstrapFontAwesomeCSS",
"Zikula\\Composer\\RequireJsConfigGenerator::regenerateRequireJs"
Expand All @@ -44,7 +44,7 @@
},
"config": {
"vendor-dir": "src/vendor",
"bin-dir": "bin",
"bin-dir": "src/bin",
"component-dir": "src/web"
},
"extra": {
Expand Down
Loading