Skip to content

Commit 3dad58a

Browse files
committed
Merge pull request #65 from RandomAPI/genderFix
Fixed gender bug if gender isn't in inc list
2 parents 7a07053 + 4d1bc5e commit 3dad58a

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

api/1.0/api.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ Generator.prototype.generate = function(cb) {
9393
}
9494
inject = injects[version][nat];
9595

96-
this.include('gender', this.gender === null ? randomItem(['male', 'female']) : this.gender);
96+
current.gender = this.gender === null ? randomItem(['male', 'female']) : this.gender;
9797

9898
var name = this.randomName(current.gender, nat);
9999
this.include('name', {
@@ -142,6 +142,11 @@ Generator.prototype.generate = function(cb) {
142142

143143
this.include('nat', nat);
144144

145+
// Gender hack - Remove gender if the user doesn't want it in the results
146+
if (this.inc.indexOf('gender') === -1) {
147+
delete current.gender;
148+
}
149+
145150
output.push(current);
146151
}
147152

0 commit comments

Comments
 (0)