Merge lp:~therp-nl/openobject-addons/lp1135873_7.0 into lp:openobject-addons/7.0

Proposed by Holger Brunn (Therp)
Status: Rejected
Rejected by: Martin Trigaux (OpenERP)
Proposed branch: lp:~therp-nl/openobject-addons/lp1135873_7.0
Merge into: lp:openobject-addons/7.0
Diff against target: 12 lines (+1/-1)
1 file modified
web_linkedin/static/src/js/linkedin.js (+1/-1)
To merge this branch: bzr merge lp:~therp-nl/openobject-addons/lp1135873_7.0
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+151031@code.launchpad.net

Commit message

[FIX] pull linkedin-api from the protocol we're on

To post a comment you must log in.
Revision history for this message
Martin Trigaux (OpenERP) (mat-openerp) wrote :

We have already merged a fix for this issue at revision 8870 (always using https) and I will then reject this patch but thanks for your work.

Regards

Unmerged revisions

8780. By Holger Brunn (Therp)

[FIX] pull linkedin-api from the protocol we're on

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'web_linkedin/static/src/js/linkedin.js'
2--- web_linkedin/static/src/js/linkedin.js 2012-11-15 15:09:45 +0000
3+++ web_linkedin/static/src/js/linkedin.js 2013-02-28 15:25:27 +0000
4@@ -19,7 +19,7 @@
5 return self.linkedin_def.promise();
6 var tag = document.createElement('script');
7 tag.type = 'text/javascript';
8- tag.src = "http://platform.linkedin.com/in.js";
9+ tag.src = document.location.protocol + "//platform.linkedin.com/in.js";
10 tag.innerHTML = 'api_key : ' + self.api_key + '\nauthorize : true\nscope: r_network r_contactinfo';
11 document.getElementsByTagName('head')[0].appendChild(tag);
12 self.linkedin_added = true;