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

Update Module.php #8

Open
wants to merge 43 commits into
base: master
Choose a base branch
from
Open

Update Module.php #8

wants to merge 43 commits into from

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: [email protected] and password admin123
2. Member user with email: [email protected] 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.

None yet

2 participants