Skip to content

Commit 96a5e19

Browse files
committed
Merge pull request #33 from codebendercc/sensiolabs-insight-fixes
Fixing SensioLabs' Insight warnings to improve the quality of the code
2 parents 79f9b09 + ffb82c6 commit 96a5e19

Some content is hidden

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

41 files changed

+1621
-2166
lines changed

Symfony/app/.htaccess

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1-
deny from all
1+
deny from all
2+

Symfony/app/AppKernel.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ public function registerBundles()
2020
);
2121

2222
if (in_array($this->getEnvironment(), array('dev', 'test'))) {
23-
$bundles[] = new Acme\DemoBundle\AcmeDemoBundle();
2423
$bundles[] = new Symfony\Bundle\WebProfilerBundle\WebProfilerBundle();
2524
$bundles[] = new Sensio\Bundle\DistributionBundle\SensioDistributionBundle();
2625
$bundles[] = new Sensio\Bundle\GeneratorBundle\SensioGeneratorBundle();

Symfony/app/config/routing_dev.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,3 @@ _configurator:
1212

1313
_main:
1414
resource: routing.yml
15-
16-
# AcmeDemoBundle routes (to be removed)
17-
_acme_demo:
18-
resource: "@AcmeDemoBundle/Resources/config/routing.yml"

Symfony/app/config/security.yml

Lines changed: 3 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -1,39 +1,8 @@
11
security:
2-
encoders:
3-
Symfony\Component\Security\Core\User\User: plaintext
4-
5-
role_hierarchy:
6-
ROLE_ADMIN: ROLE_USER
7-
ROLE_SUPER_ADMIN: [ROLE_USER, ROLE_ADMIN, ROLE_ALLOWED_TO_SWITCH]
2+
firewalls:
3+
anonymous:
4+
anonymous: ~
85

96
providers:
107
in_memory:
118
memory:
12-
users:
13-
user: { password: userpass, roles: [ 'ROLE_USER' ] }
14-
admin: { password: adminpass, roles: [ 'ROLE_ADMIN' ] }
15-
16-
firewalls:
17-
dev:
18-
pattern: ^/(_(profiler|wdt)|css|images|js)/
19-
security: false
20-
21-
login:
22-
pattern: ^/demo/secured/login$
23-
security: false
24-
25-
secured_area:
26-
pattern: ^/demo/secured/
27-
form_login:
28-
check_path: _security_check
29-
login_path: _demo_login
30-
logout:
31-
path: _demo_logout
32-
target: _demo
33-
#anonymous: ~
34-
#http_basic:
35-
# realm: "Secured Demo Area"
36-
37-
access_control:
38-
- { path: ^/demo/secured/hello/admin/, roles: ROLE_ADMIN }
39-
#- { path: ^/login, roles: IS_AUTHENTICATED_ANONYMOUSLY, requires_channel: https }

Symfony/src/.htaccess

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1-
deny from all
1+
deny from all
2+

Symfony/src/Acme/DemoBundle/AcmeDemoBundle.php

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

Symfony/src/Acme/DemoBundle/Controller/DemoController.php

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

Symfony/src/Acme/DemoBundle/Controller/SecuredController.php

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

Symfony/src/Acme/DemoBundle/Controller/WelcomeController.php

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

Symfony/src/Acme/DemoBundle/DependencyInjection/AcmeDemoExtension.php

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

0 commit comments

Comments
 (0)