Skip to content

Commit ca53a75

Browse files
committed
2018-02-26 AC: Merges branch '2.2.0-codefreeze'.
2 parents 5093022 + 0ee216b commit ca53a75

File tree

147 files changed

+4035
-6881
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

147 files changed

+4035
-6881
lines changed

.codeclimate.yml

Lines changed: 64 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,40 @@
11
---
2+
version: "2" # required to adjust maintainability checks
3+
checks:
4+
argument-count:
5+
config:
6+
threshold: 15
7+
complex-logic:
8+
config:
9+
threshold: 15
10+
file-lines:
11+
config:
12+
threshold: 1500
13+
method-complexity:
14+
config:
15+
threshold: 50
16+
method-count:
17+
config:
18+
threshold: 50
19+
method-lines:
20+
config:
21+
threshold: 150
22+
nested-control-flow:
23+
config:
24+
threshold: 4
25+
return-statements:
26+
config:
27+
threshold: 5
28+
similar-code:
29+
config:
30+
threshold: # language-specific defaults. an override will affect all languages.
31+
identical-code:
32+
config:
33+
threshold: # language-specific defaults. an override will affect all languages.
34+
235
engines:
336
duplication:
4-
enabled: true
37+
enabled: false
538
config:
639
languages:
740
php:
@@ -10,19 +43,38 @@ engines:
1043
enabled: true
1144
config:
1245
standard: PSR2
46+
checks:
47+
PSR1 Files SideEffects FoundWithSymbols:
48+
enabled: false
49+
PSR1 Classes ClassDeclaration MissingNamespace:
50+
enabled: false
51+
Squiz Classes ValidClassName NotCamelCaps:
52+
enabled: false
1353

1454
ratings:
1555
paths:
1656
- "**.php"
1757

18-
exclude_paths:
19-
- "bin/"
20-
- "build/"
21-
- "docs/"
22-
- "docs-api/"
23-
- "language_files/"
24-
- "lib/"
25-
- "sef/"
26-
- "tests/"
27-
- "tmpl/"
28-
- "vendor/"
58+
exclude_patterns:
59+
- "config/"
60+
- "db/"
61+
- "dist/"
62+
- "features/"
63+
- "**/node_modules/"
64+
- "script/"
65+
- "**/spec/"
66+
- "**/test/"
67+
- "**/tests/"
68+
- "**/vendor/"
69+
- "**/*_test.go"
70+
- "**/*.d.ts"
71+
- "bin/"
72+
- "build/"
73+
- "data/"
74+
- "docs/"
75+
- "docs-api/"
76+
- "language_files/"
77+
- "lib/"
78+
- "sef/"
79+
- "tests/"
80+
- "tmpl/"

.travis.yml

Lines changed: 57 additions & 50 deletions
Original file line numberDiff line numberDiff line change
@@ -1,69 +1,76 @@
1-
branches:
2-
except:
3-
- 1.8.9
4-
- 1.8.8
5-
- 1.8.7
6-
- develop
1+
env:
2+
global:
3+
- SAUCE_USERNAME=andrewscaya
4+
- secure: "Cwt3tdw/XN4MX/rLxCh23Af6y6GOKccX/YTPCQUAJDkKvUDqPtM/Q0PnCQwHdOfhmliFsP9QVZzrv3cZrpggNrjU5cctc+w9GsdMogZlEULaV3sa4DgXHjfrTniguOh2YsZQ3tcTQkL/jH3mJ6FiY+8zZOGJvMEydyFp5u06khW/ahmot6EVkXt6+4dswtfckTaQZiXBdOdjyZ9lMbnjEpeLp7Ed0FqQDdq9GZK0M9weurEHZwLhClXzyMhHMA4GolvYCFR9+a1WBU3T/c3GJjMfVLjqvW3l5MzI1/ShFeC8/mXaaWfzNobo+x6Iv3CNUB657dRBxFr2PFvIx8GQYZXG1q7WBUrONoXZFUqEdUYHMTly/vGYkiMXfXYJAHpAuRmEHTyprYFDc7TtnPGC3bq5eqmUwZlE3Sjm0HnDQWhwpzVr0yTqccr3cXPNBrK9xLMTLo0sbGHRMvJLt7LltNd1dkuRNoo5sPedYK46pZQLaJ5sKWCHMNBBnVVHXJzjdaoRdFk2NY2Y9E8PMiqRl68kHl2hs7pSUu+QhrC0bPLGcf8cVxko/foPAbXDYQalG+T1zsbrPHhb5ZvE8Un3IZF6Gpl1QeYqbD0HWbm/chHy+0oXiIvRAESxoQtv505if1u8NszHYZVL0+Cj/dkKunwhZoIhr+DHsGyAsfE8RnU="
5+
- secure: "fatMNhRKgFgwcE0AY1kcuL/hC7698iOta96kotcpBT4kCrIJ0HjlhVWzh94JzSYl0MaiSU0hgi/KG1Goo/8byEbkB3TYrw/gasJLontbl6aCVPNi/HRS4uD2kvB6NzqNzkvX9zd4L/MjZzOJc0CGR/EbO/gVhXYqodL8Qbj5Kdwlodx7Aa4CIC0vF+WiwArQWqIFWPoRG81dgxd0Ze95fOQwLYfSuwmm1IWor42oZLR4UnAEtIDDcuCVKEeADgPWsTIUZP+05KhiMhE8sGV2kolp4FbG8yqFsGe06DfGTnCujFdPZINobEip7HmKhZQWmgC9/9QJLW1G+jcA2cwugk4/1hz0eiuFbVZcElY3yISwNOv3UrQX49FOZ83AiueqwODfhdaGBgVXStSj0f2lKPixRD5M/0dovHT0qDKB0C/OnBYDXtUW84U+dBi2LlQKRFZn5XJ2uDYGWnTsMIAbtttxdmMDg/9GXFWY+/5PXzTaD38+V+C1BGxI4/xfTk8Pm8sLJef08vm7apPv/x4k3YaBxIxVbXfg9UgN2mYcaQxJBMCOHxiRr77gHqmPmEAtRCVFWJ+t/e8UcCRReD2gYZ/WmTRnTz5YmaaaySA5yr+KpwRL/UczrY4R/0KxJnOONiBl3z6UM0f/ysufaAE7bAvr07YEm3IrYVUAsVHV0Ak="
76

87
sudo: false
98

109
language: php
1110

1211
cache:
13-
directories:
14-
- $HOME/.composer/cache
15-
- vendor
12+
directories:
13+
- $HOME/.composer/cache
14+
- vendor
1615

1716
services:
1817
- mysql
1918

2019
addons:
21-
firefox: "31.6"
20+
sauce_connect: true
21+
22+
branches:
23+
except:
24+
- 1.8.9
25+
- 1.8.8
26+
- 1.8.7
27+
- develop
2228

2329
matrix:
24-
fast_finish: true
25-
include:
26-
- php: 5.5
27-
env:
28-
- COMPOSER_PROCESS_TIMEOUT=1200
29-
- TEST_CHECK=true
30-
- CS_CHECK=true
31-
- CC_CHECK=false
32-
- php: 5.6
33-
env:
34-
- COMPOSER_PROCESS_TIMEOUT=1200
35-
- TEST_CHECK_COVER=true
36-
- CS_CHECK=true
37-
- CC_CHECK=false
38-
- CODECLIMATE_REPO_TOKEN=cdf9d20bb5113bdc18f79b722ca5a54c965e2ccae2912ba40d92d0651cf8affa
39-
- php: 7
40-
env:
41-
- TEST_CHECK=false
42-
- CS_CHECK=true
43-
- CC_CHECK=false
44-
- php: hhvm
45-
allow_failures:
46-
- php: hhvm
47-
30+
fast_finish: true
31+
include:
32+
- php: 5.5
33+
env:
34+
- COMPOSER_PROCESS_TIMEOUT=1200
35+
- TEST_CHECK=true
36+
- CS_CHECK=true
37+
- CC_CHECK=true
38+
- php: 5.6
39+
env:
40+
- COMPOSER_PROCESS_TIMEOUT=1200
41+
- TEST_CHECK_COVER=true
42+
- CS_CHECK=true
43+
- CC_CHECK=true
44+
- php: 7
45+
env:
46+
- TEST_CHECK=false
47+
- CS_CHECK=true
48+
- CC_CHECK=true
49+
- php: hhvm
50+
env:
51+
- TEST_CHECK=false
52+
- CS_CHECK=true
53+
- CC_CHECK=false
54+
allow_failures:
55+
- php: hhvm
56+
4857
before_install:
49-
# XVFB
50-
- "export DISPLAY=:99.0"
51-
- "sh -e /etc/init.d/xvfb start"
52-
- sleep 5 # give xvfb some time to start
53-
- travis_retry composer self-update
54-
- if [[ $TEST_CHECK_54 == 'true' ]]; then rm $TRAVIS_BUILD_DIR/composer.json && mv $TRAVIS_BUILD_DIR/composer.php54.json $TRAVIS_BUILD_DIR/composer.json ; fi
55-
- if [[ -z "$TEST_CHECK_COVER" ]]; then phpenv config-rm xdebug.ini || return 0 ; fi
56-
- if [[ $CC_CHECK == 'true' ]]; then $TRAVIS_BUILD_DIR/bin/codeception-setup.sh ; fi
57-
- sleep 5 # gives enough time to Selenium to start
58+
- travis_retry composer self-update
59+
- if [[ $TEST_CHECK_54 == 'true' ]]; then rm $TRAVIS_BUILD_DIR/composer.json && mv $TRAVIS_BUILD_DIR/composer.php54.json $TRAVIS_BUILD_DIR/composer.json ; fi
60+
- if [[ -z "$TEST_CHECK_COVER" ]]; then phpenv config-rm xdebug.ini || return 0 ; fi
5861

5962
install:
60-
- travis_retry composer install --no-interaction
61-
63+
- travis_retry composer install --no-interaction
64+
65+
before_script:
66+
- if [[ $CC_CHECK == 'true' ]]; then $TRAVIS_BUILD_DIR/bin/selenium-setup.sh ; fi
67+
- if [[ $CC_CHECK == 'true' ]]; then npm install -g se-interpreter ; fi
68+
6269
script:
63-
- if [[ $TEST_CHECK == 'true' || $TEST_CHECK_54 == 'true' ]]; then composer test ; fi
64-
- if [[ $TEST_CHECK_COVER == 'true' ]]; then composer test-cover ; fi
65-
- if [[ $CS_CHECK == 'true' ]]; then composer cs ; fi
66-
- if [[ $CC_CHECK == 'true' ]]; then composer cc ; fi
70+
- if [[ $CC_CHECK == 'true' ]]; then se-interpreter tests/selenium/se-interpreter_config.json ; fi
71+
- if [[ $TEST_CHECK == 'true' || $TEST_CHECK_54 == 'true' ]]; then composer test ; fi
72+
- if [[ $TEST_CHECK_COVER == 'true' ]]; then composer test-cover ; fi
73+
- if [[ $CS_CHECK == 'true' ]]; then composer cs ; fi
6774

6875
after_success:
69-
- if [[ $TEST_CHECK_COVER == 'true' ]]; then composer test-report ; fi
76+
- if [[ $TEST_CHECK_COVER == 'true' ]]; then composer test-report ; fi

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
# CHANGELOG
22

3+
## 2.2.0 (2018-02-23)
4+
5+
- Fixes issue #16 - Language overrides do not work in SEF 1.8.9 and Joomla 3.6.5.
6+
- Fixes all functional tests by having them run on the Sauce Labs Selenium Server.
7+
- Fixes a minor issue with the upload error messages.
8+
39
## 2.1.0 (2017-12-22)
410

511
- Fixes issue #15 - Allow users to configure an automated reply email to those submitting the form.

README.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,5 +7,4 @@ Basic email contact form which lets you define up to eight (8) configurable fiel
77
[![Test Coverage](https://codeclimate.com/github/andrewscaya/mod_simpleemailform/badges/coverage.svg)](https://codeclimate.com/github/andrewscaya/mod_simpleemailform)
88
[![Issue Count](https://codeclimate.com/github/andrewscaya/mod_simpleemailform/badges/issue_count.svg)](https://codeclimate.com/github/andrewscaya/mod_simpleemailform)
99
[![PDS Skeleton](https://img.shields.io/badge/pds-skeleton-blue.svg?style=flat-square)](https://github.com/php-pds/skeleton)
10-
11-
Code coverage: SEFv2 100%.
10+
[![Documentation Status](https://readthedocs.org/projects/simple-email-form/badge/?version=stable)](http://simple-email-form.readthedocs.io/en/stable/?badge=stable)

bin/codeception-setup.sh

Lines changed: 0 additions & 16 deletions
This file was deleted.

bin/selenium-setup.sh

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
#!/usr/bin/env bash
2+
cd /tmp
3+
#wget http://selenium-release.storage.googleapis.com/2.53/selenium-server-standalone-2.53.1.jar
4+
#java -Xms40m -Xmx256m -jar /tmp/selenium-server-standalone-2.53.1.jar > /dev/null 2>&1 &
5+
wget -O joomla_3-8-5-stable-full_package.zip https://downloads.joomla.org/cms/joomla3/3-8-5/Joomla_3-8-5-Stable-Full_Package.zip?format=zip
6+
wget -O mod_simpleemailform-latest.tar.gz https://codeload.github.com/andrewscaya/mod_simpleemailform/tar.gz/latest
7+
mkdir joomla
8+
cd joomla
9+
unzip ../joomla_3-8-5-stable-full_package.zip
10+
mysql -uroot -e "CREATE DATABASE joomlatest;"
11+
(php -S localhost:8181 > /dev/null 2>&1) &
12+
exit 0

codeception.yml

Lines changed: 0 additions & 21 deletions
This file was deleted.

composer.json

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,6 @@
4040
"codeclimate/php-test-reporter": "0.2.0",
4141
"satooshi/php-coveralls": "~0.6",
4242
"squizlabs/php_codesniffer": "^2.3",
43-
"se/selenium-server-standalone": "^2.53",
44-
"codeception/codeception": "^2.2",
45-
"joomla-projects/joomla-browser": "v3.6.5.1",
4643
"phpdocumentor/phpdocumentor": "2.9.0",
4744
"pds/skeleton": "~1.0"
4845
},
@@ -57,23 +54,20 @@
5754
"./tests/unit"
5855
],
5956
"exclude-from-classmap": [
60-
"./vendor/codeception",
6157
"./vendor/joomla-projects",
6258
"./vendor/consolidation"
6359
]
6460
},
6561
"scripts": {
6662
"check": [
6763
"@cs",
68-
"@test",
69-
"@cc"
64+
"@test"
7065
],
7166
"cs": "phpcs",
7267
"cs-fix": "phpcbf",
7368
"test": "phpunit",
7469
"test-cover": "phpunit --coverage-clover build/logs/clover.xml",
7570
"test-report": "test-reporter --coverage-report build/logs/clover.xml",
76-
"cc": "codecept run acceptance",
7771
"apidocs": "phpdoc --template=responsive-twig"
7872
}
7973
}

composer.phar

7.11 KB
Binary file not shown.

docs-api/classes/sefv2customrenderinginterface.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -456,7 +456,7 @@ <h3>Response</h3>
456456
Template is built using <a href="http://twitter.github.com/bootstrap/">Twitter Bootstrap 2</a> and icons provided by
457457
<a href="http://glyphicons.com/">Glyphicons</a>.<br/>
458458
Documentation is powered by <a href="http://www.phpdoc.org/">phpDocumentor </a> and<br/>
459-
generated on Sat, 16 Dec 2017 15:42:10 -0400.<br/>
459+
generated on Fri, 23 Feb 2018 20:05:31 -0400.<br/>
460460
</footer>
461461
</body>
462462
</html>

0 commit comments

Comments
 (0)