Skip to content

Commit fd7d9f6

Browse files
committed
impr: changed "undefined" as default value for unused attributes in models (close #5)
1 parent a2b3f5d commit fd7d9f6

2 files changed

Lines changed: 31 additions & 31 deletions

File tree

dist/aping-plugin-github.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/aping-github-helper.js

Lines changed: 30 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -91,17 +91,17 @@ jjtApingGithub.service('apingGithubHelper', ['apingModels', 'apingTimeHelper', '
9191
var repoObject = apingModels.getNew("repo", this.getThisPlattformString());
9292

9393
$.extend(true, repoObject, {
94-
owner_name : _item.owner ? _item.owner.login : false,
95-
owner_id : _item.owner ? _item.owner.id : false,
96-
owner_link : _item.owner ? _item.owner.html_url : false,
97-
owner_img_url : _item.owner ? _item.owner.avatar_url : false,
94+
owner_name : _item.owner ? _item.owner.login : undefined,
95+
owner_id : _item.owner ? _item.owner.id : undefined,
96+
owner_link : _item.owner ? _item.owner.html_url : undefined,
97+
owner_img_url : _item.owner ? _item.owner.avatar_url : undefined,
9898
name : _item.name,
9999
id: _item.id,
100100
fullname: _item.full_name,
101-
description : _item.description || false,
101+
description : _item.description || undefined,
102102
url : _item.html_url,
103-
homepage : _item.homepage || false,
104-
language : _item.language || false,
103+
homepage : _item.homepage || undefined,
104+
language : _item.language || undefined,
105105
clone_url : _item.clone_url,
106106
git_url : _item.git_url,
107107
ssh_url : _item.ssh_url,
@@ -113,10 +113,10 @@ jjtApingGithub.service('apingGithubHelper', ['apingModels', 'apingTimeHelper', '
113113
forks : _item.forks_count,
114114
created_timestamp : apingTimeHelper.getTimestampFromDateString(_item.created_at, 1000, 3600*1000),
115115
created_date_time: new Date(_item.created_at),
116-
updated_timestamp: _item.updated_at ? apingTimeHelper.getTimestampFromDateString(_item.updated_at, 1000, 3600*1000) : false,
117-
updated_date_time: _item.updated_at ? new Date(_item.updated_at) : false,
118-
pushed_timestamp: _item.pushed_at ? apingTimeHelper.getTimestampFromDateString(_item.pushed_at, 1000, 3600*1000) : false,
119-
pushed_date_time: _item.pushed_at ? new Date(_item.pushed_at) : false,
116+
updated_timestamp: _item.updated_at ? apingTimeHelper.getTimestampFromDateString(_item.updated_at, 1000, 3600*1000) : undefined,
117+
updated_date_time: _item.updated_at ? new Date(_item.updated_at) : undefined,
118+
pushed_timestamp: _item.pushed_at ? apingTimeHelper.getTimestampFromDateString(_item.pushed_at, 1000, 3600*1000) : undefined,
119+
pushed_date_time: _item.pushed_at ? new Date(_item.pushed_at) : undefined,
120120
});
121121

122122
return repoObject;
@@ -126,28 +126,28 @@ jjtApingGithub.service('apingGithubHelper', ['apingModels', 'apingTimeHelper', '
126126
var activityObject = apingModels.getNew("activity", this.getThisPlattformString());
127127
128128
$.extend(true, activityObject, {
129-
body : false,
129+
body : undefined,
130130
131-
actor_name : _item.actor ? _item.actor.login : false, //who?
132-
actor_id : _item.actor ? _item.actor.id : false,
133-
actor_url : _item.actor ? this.getThisPlattformLink()+_item.actor.login : false,
134-
actor_img_url : _item.actor ? _item.actor.avatar_url : false,
135-
actor_type: false,
131+
actor_name : _item.actor ? _item.actor.login : undefined, //who?
132+
actor_id : _item.actor ? _item.actor.id : undefined,
133+
actor_url : _item.actor ? this.getThisPlattformLink()+_item.actor.login : undefined,
134+
actor_img_url : _item.actor ? _item.actor.avatar_url : undefined,
135+
actor_type: undefined,
136136
137-
//action_name : false,
138-
//action_message : false,
137+
//action_name : undefined,
138+
//action_message : undefined,
139139
action_id : _item.id,
140-
//action_url : false,
141-
//action_img : false,
140+
//action_url : undefined,
141+
//action_img : undefined,
142142
action_type: _item.type,
143143
144-
object_name : _item.repo ? _item.repo.name : false,
145-
object_id : _item.repo ? _item.repo.id : false,
146-
object_img : false,
147-
object_url : _item.repo ? this.getThisPlattformLink()+_item.repo.name : false,
148-
object_type: _item.repo ? "repository" : false,
144+
object_name : _item.repo ? _item.repo.name : undefined,
145+
object_id : _item.repo ? _item.repo.id : undefined,
146+
object_img : undefined,
147+
object_url : _item.repo ? this.getThisPlattformLink()+_item.repo.name : undefined,
148+
object_type: _item.repo ? "repository" : undefined,
149149
150-
//context : false,
150+
//context : undefined,
151151
timestamp : apingTimeHelper.getTimestampFromDateString(_item.created_at, 1000, 3600*1000),
152152
date_time: new Date(_item.created_at),
153153
@@ -165,9 +165,9 @@ jjtApingGithub.service('apingGithubHelper', ['apingModels', 'apingTimeHelper', '
165165
this.getActionMessageByTypeAndPayload = function (_type, _payload) {
166166
167167
var returnObject ={
168-
name : false,
168+
name : undefined,
169169
message : "",
170-
url : false,
170+
url : undefined,
171171
};
172172
173173
switch(_type) {
@@ -195,7 +195,7 @@ jjtApingGithub.service('apingGithubHelper', ['apingModels', 'apingTimeHelper', '
195195
}
196196
197197
if(returnObject.message === "") {
198-
returnObject.message = false;
198+
returnObject.message = undefined;
199199
}
200200
201201
return returnObject;

0 commit comments

Comments
 (0)