Skip to content

Update Module.php#8

Open
davidmintz wants to merge 43 commits intoslaff:masterfrom
davidmintz:patch-1
Open

Update Module.php#8
davidmintz wants to merge 43 commits intoslaff:masterfrom
davidmintz:patch-1

Conversation

@davidmintz
Copy link

it seems you no longer need to get the $auth from the service manager, since you're getting a $currentUser whether authenticated or not. Hence the proposed deletion of the line, since it may tend to confuse (it kind of confused me :-) ).

Slavey Karadzhov added 30 commits March 20, 2015 13:11
tags:[ch-reusability-entity]
…nd build the form using the annotation builder.
tags:[ch-authentication]
tags:[ch-exam-skeleton]
…I: /exam/test/take/1.

tags:[ch-exam-take1]
Slavey Karadzhov and others added 13 commits April 9, 2015 14:35
tags:[ch-cache-actioncache]
…mespaces and

Zend Framework 2.
Enjoy and happy Zending!

tags:[ch-final-words]

Make sure to read also the information below:

Ready to test version with prefilled SQLite database.
In order to use it make sure that:

You have read the README.md file and the data/db/tc.sqlite file is readable from the web server user.
The following commands can help you set the group to www-data and adjust the permissions.
chgrp www-data -R data/db
chmod g+rx data/db
chmod g+w data/db/tc.sqlite

There are two users that you can use.
1. Admin user with email: admin@learnzf2.com and password admin123
2. Member user with email: member@learnzf2.com and password member123.

If you want to take some of the sample exams that come with this application then
you have to login and go to Exam -> List and choose one from the list.
it seems you no longer need to get the $auth from the service manager, since you're getting a $currentUser whether authenticated or not. Hence the proposed deletion of that line. It may tend to confuse (it kind of confused me :-) ).
@slaff
Copy link
Owner

slaff commented Sep 4, 2015

I should probably add also some information why the auth service is removed from the code.. Have to think about a change that does not require a change in the content of the book.

@slaff slaff force-pushed the master branch 3 times, most recently from 30589a6 to 192a9b1 Compare November 9, 2015 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants