Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: proxy-less env support #171

Closed
wants to merge 6 commits into from
Closed

fix: proxy-less env support #171

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
311c075
Proxy-less env support
bassarisse Nov 7, 2018
65eb8df
chore: build
May 11, 2020
38fe004
chore: remove redundant repetition
May 12, 2020
3a6255e
chore: readability
May 12, 2020
6a34e9a
chore: explicit return
May 14, 2020
57249c9
chore: addRuleSet return
May 14, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
10 changes: 5 additions & 5 deletions src/v8n.js
View file
Open in desktop
Expand Up @@ -40,16 +40,16 @@ function proxyContext(context) {
}

function proxyContextEs5(context) {
const buildRuleSet = ruleSet =>
const addRuleSet = (ruleSet, targetContext) =>
Object.keys(ruleSet).forEach(prop => {
context[prop] = (...args) => {
const newContext = proxyContextEs5(context._clone());
targetContext[prop] = (...args) => {
const newContext = proxyContextEs5(targetContext._clone());
return newContext._applyRule(ruleSet[prop], prop)(...args);
};
});
sbarfurth marked this conversation as resolved.
Show resolved Hide resolved

buildRuleSet(availableRules);
buildRuleSet(customRules);
addRuleSet(availableRules, context);
Copy link
Collaborator

Choose a reason for hiding this comment

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

The returned, changed context should be reassigned here. otherwise the context variable is changed below and it's not clear where. So it should be clearly reassigned.

addRuleSet(customRules, context);

Object.keys(availableModifiers).forEach(prop => {
Object.defineProperty(context, prop, {
Expand Down