Skip to content

[OPENMEETINGS-2359] check if data is available#88

Merged
solomax merged 1 commit intoapache:masterfrom
CkNoSFeRaTU:getpeer-undefined
May 17, 2020
Merged

[OPENMEETINGS-2359] check if data is available#88
solomax merged 1 commit intoapache:masterfrom
CkNoSFeRaTU:getpeer-undefined

Conversation

@CkNoSFeRaTU
Copy link
Copy Markdown
Contributor

No description provided.

const w = $('#' + VideoUtil.getVid(m.uid))
, v = w.data()
, peer = v.getPeer();
, v = w.data();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer shorter version: , peer = v && v.getPeer();

, v = w.data()
, peer = v.getPeer();
, v = w.data();
var peer;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

@solomax solomax merged commit e63c565 into apache:master May 17, 2020
@solomax
Copy link
Copy Markdown
Contributor

solomax commented May 17, 2020

Thanks for the contribution!

@CkNoSFeRaTU CkNoSFeRaTU deleted the getpeer-undefined branch May 18, 2020 12:04
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