Skip to content

Commit 08c5330

Browse files
committed
refactor: working after move
1 parent 4abb2f6 commit 08c5330

File tree

6 files changed

+9
-536
lines changed

6 files changed

+9
-536
lines changed

gulpfile.js/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -645,7 +645,7 @@ function inlineTextRequire(file, content, srcDir, isDevBuild = true) {
645645
const requireFragments = extractRequireTextFragments(content);
646646
for (const {requirePath, requireStatement} of requireFragments) {
647647
let filePath = srcDir + requirePath;
648-
if(requirePath.startsWith("./")) {
648+
if(requirePath.startsWith("./") || requirePath.startsWith("../")) {
649649
filePath = path.join(path.dirname(file), requirePath);
650650
}
651651
let textContent = textContentMap[getKey(filePath, isDevBuild)];
@@ -663,7 +663,7 @@ function inlineTextRequire(file, content, srcDir, isDevBuild = true) {
663663
textContentMap[getKey(filePath, isDevBuild)] = fileContent;
664664
textContent = fileContent;
665665
}
666-
if((requirePath.endsWith(".js") && !requirePath.includes("./")) // js files that are relative paths are ok
666+
if((requirePath.endsWith(".js") && !requirePath.includes("./") && !requirePath.includes("../")) // js files that are relative paths are ok
667667
|| excludeSuffixPathsInlining.some(ext => requirePath.endsWith(ext))) {
668668
console.warn("Not inlining JS/JSON file:", requirePath, filePath);
669669
if(filePath.includes("phoenix-pro")) {

src/brackets.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -139,8 +139,6 @@ define(function (require, exports, module) {
139139
require("widgets/InlineMenu");
140140
require("thirdparty/tinycolor");
141141
require("utils/LocalizationUtils");
142-
require("services/login-desktop");
143-
require("services/login-browser");
144142

145143
// DEPRECATED: In future we want to remove the global CodeMirror, but for now we
146144
// expose our required CodeMirror globally so as to avoid breaking extensions in the

src/extensionsIntegrated/Phoenix-live-preview/main.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,6 @@ define(function (require, exports, module) {
7373
panelHTML = require("text!./panel.html"),
7474
Dialogs = require("widgets/Dialogs"),
7575
DefaultDialogs = require("widgets/DefaultDialogs"),
76-
ProDialogs = require("services/pro-dialogs"),
7776
utils = require('./utils');
7877

7978
const KernalModeTrust = window.KernalModeTrust;
@@ -411,7 +410,12 @@ define(function (require, exports, module) {
411410
LiveDevelopment.setMode(LiveDevelopment.CONSTANTS.LIVE_HIGHLIGHT_MODE);
412411
} else if (index === 2) {
413412
if (!LiveDevelopment.setMode(LiveDevelopment.CONSTANTS.LIVE_EDIT_MODE)) {
414-
ProDialogs.showProUpsellDialog(ProDialogs.UPSELL_TYPE_LIVE_EDIT);
413+
if(KernalModeTrust.ProDialogs) {
414+
KernalModeTrust.ProDialogs.showProUpsellDialog(
415+
KernalModeTrust.ProDialogs.UPSELL_TYPE_LIVE_EDIT);
416+
} else {
417+
Metrics.countEvent(Metrics.EVENT_TYPE.PRO, "proUpsellDlg", "fail");
418+
}
415419
}
416420
} else if (item === Strings.LIVE_PREVIEW_EDIT_HIGHLIGHT_ON) {
417421
// Don't allow edit highlight toggle if edit features are not active

src/help/HelpCommandHandlers.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ define(function (require, exports, module) {
3131
NativeApp = require("utils/NativeApp"),
3232
Strings = require("strings"),
3333
StringUtils = require("utils/StringUtils"),
34-
ManageLicenses = require("services/manage-licenses"),
34+
ManageLicenses = require("extensionsIntegrated/phoenix-pro/services/manage-licenses"),
3535
AboutDialogTemplate = require("text!htmlContent/about-dialog.html"),
3636
ContributorsTemplate = require("text!htmlContent/contributors-list.html"),
3737
Mustache = require("thirdparty/mustache/mustache");

test/UnitTestSuite.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,6 @@ define(function (require, exports, module) {
124124
require("spec/login-desktop-integ-test");
125125
require("spec/LocalizationUtils-test");
126126
require("spec/ScrollTrackHandler-integ-test");
127-
require("spec/login-utils-test");
128127
// Integrated extension tests
129128
require("spec/Extn-InAppNotifications-integ-test");
130129
require("spec/Extn-RemoteFileAdapter-integ-test");

0 commit comments

Comments
 (0)