Skip to content

Commit 6d7838d

Browse files
committed
register plugin has 100% coverage. the hard part is #done
1 parent 73e4cf8 commit 6d7838d

1 file changed

Lines changed: 3 additions & 8 deletions

File tree

lib/register_handler.js

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -12,16 +12,11 @@ module.exports = function register_handler(request, reply, source, error) {
1212
if (request.method === 'get') { // get does not send payload so return reg
1313
return reply.view('index', { title : 'Please Register' }).code(200);
1414
}
15-
if(!request.payload || request.payload && error) { // *any* validation errors
16-
var errors, values; // return empty if not set.
17-
if(error && error.data) { // means the handler is dual-purpose
18-
errors = help.extract_validation_error(error); // error field & message
19-
values = help.return_form_input_values(error); // avoid wiping form data
20-
}
15+
if(!request.payload || request.payload && error && error.data) { // joi error
2116
return reply.view('index', {
2217
title : 'Please Register ' + request.server.version,
23-
error : errors, // error object used in html template
24-
values : values // (escaped) values displayed in form inputs
18+
errors : help.extract_validation_error(error), // error field & message
19+
values : help.return_form_input_values(error) // avoid wiping form data
2520
}).code(400);
2621
} // this block is essentially doing *manual* Joi validation to show html!
2722
else { // the payload was valid, lets see if the person has already registered

0 commit comments

Comments
 (0)