Merge branch 'karolyi-master'

This commit is contained in:
Philipp Zins 2014-05-13 07:32:13 +02:00
commit 0bc284a7fd
2 changed files with 92 additions and 6 deletions

View File

@ -5,11 +5,75 @@
var beautifyJS = require('js-beautify'); var beautifyJS = require('js-beautify');
var beautifyHTML = require('js-beautify').html; var beautifyHTML = require('js-beautify').html;
var beautifyCSS = require('js-beautify').css; var beautifyCSS = require('js-beautify').css;
var fs = require('fs');
var path = require('path');
var cc = require('config-chain');
var nopt = require('nopt');
var extend = require('extend');
// TODO: Copied from jsbeautify, please update it from time to time
var knownOpts = {
// Beautifier
'indent_size': Number,
'indent_char': String,
'indent_level': Number,
'indent_with_tabs': Boolean,
'preserve_newlines': Boolean,
'max_preserve_newlines': Number,
'space_in_paren': Boolean,
'jslint_happy': Boolean,
// TODO: expand-strict is obsolete, now identical to expand. Remove in future version
'brace_style': ['collapse', 'expand', 'end-expand', 'expand-strict'],
'break_chained_methods': Boolean,
'keep_array_indentation': Boolean,
'unescape_strings': Boolean,
'wrap_line_length': Number,
'e4x': Boolean,
// HTML-only
'max_char': Number, // obsolete since 1.3.5
'unformatted': [String, Array],
'indent_inner_html': [Boolean],
'indent_scripts': ['keep', 'separate', 'normal'],
// CLI
'version': Boolean,
'help': Boolean,
'files': [path, Array],
'outfile': path,
'replace': Boolean,
'quiet': Boolean,
'type': ['js', 'css', 'html'],
'config': path
};
var Subscriber = require('emissary').Subscriber; var Subscriber = require('emissary').Subscriber;
var plugin = module.exports; var plugin = module.exports;
Subscriber.extend(plugin); Subscriber.extend(plugin);
function verifyExists(fullPath) {
return fs.existsSync(fullPath) ? fullPath : null;
}
function findRecursive(dir, fileName) {
var fullPath = path.join(dir, fileName);
var nextDir = path.dirname(dir);
var result = verifyExists(fullPath);
if (!result && (nextDir !== dir)) {
result = findRecursive(nextDir, fileName);
}
return result;
}
function getUserHome() {
return process.env.HOME || process.env.USERPROFILE;
}
function cleanOptions(data, types) {
nopt.clean(data, types);
return data;
}
function beautify() { function beautify() {
var text; var text;
var editor = atom.workspace.getActiveEditor(); var editor = atom.workspace.getActiveEditor();
@ -22,6 +86,24 @@ function beautify() {
'indent_char': softTabs ? ' ' : '\t' 'indent_char': softTabs ? ' ' : '\t'
}; };
// Look for .jsbeautifierrc in file and home path, check env variables
var editedFilePath = editor.getPath();
var cfg = cc(
cleanOptions(cc.env('jsbeautify_'), knownOpts),
editedFilePath ? findRecursive(path.dirname(editedFilePath),
'.jsbeautifyrc') : null,
verifyExists(path.join(getUserHome() || '', '.jsbeautifyrc'))
).list;
// cc(...).snapshot SHOULD contain the same what I construct below,
// however I have not the faintest idea why it doesn't work here.
// It works at js-beautify cli, but not here. Weird.
var collectedConfig = {};
for (var idx = cfg.length - 1; idx >= 0; idx--) {
collectedConfig = extend(cfg[idx], collectedConfig);
}
// Override the indenting options from the editor
beautifyOptions = extend(collectedConfig, beautifyOptions);
if (isSelection) { if (isSelection) {
text = editor.getSelectedText(); text = editor.getSelectedText();
} else { } else {
@ -54,11 +136,11 @@ function beautify() {
} }
function handleSafeEvent() { function handleSafeEvent() {
atom.workspace.eachEditor(function(editor) { atom.workspace.eachEditor(function (editor) {
var buffer = editor.getBuffer(); var buffer = editor.getBuffer();
plugin.unsubscribe(buffer); plugin.unsubscribe(buffer);
if (atom.config.get('beautify.beautifyOnSave')) { if (atom.config.get('atom-beautify.beautifyOnSave')) {
var events = 'will-be-saved'; var events = 'will-be-saved';
plugin.subscribe(buffer, events, beautify); plugin.subscribe(buffer, events, beautify);
} }
@ -69,8 +151,9 @@ plugin.configDefaults = {
beautifyOnSave: false beautifyOnSave: false
}; };
plugin.activate = function() { plugin.activate = function () {
handleSafeEvent(); handleSafeEvent();
plugin.subscribe(atom.config.observe('beautify.beautifyOnSave', handleSafeEvent)); plugin.subscribe(atom.config.observe('atom-beautify.beautifyOnSave',
handleSafeEvent));
return atom.workspaceView.command('beautify', beautify); return atom.workspaceView.command('beautify', beautify);
}; };

View File

@ -1,7 +1,7 @@
{ {
"name": "atom-beautify", "name": "atom-beautify",
"main": "./lib/atom-beautify", "main": "./lib/atom-beautify",
"version": "0.2.2", "version": "0.2.3",
"private": true, "private": true,
"description": "Beautify HTML, CSS and Javascript in Atom", "description": "Beautify HTML, CSS and Javascript in Atom",
"repository": "https://github.com/donaldpipowitch/atom-beautify", "repository": "https://github.com/donaldpipowitch/atom-beautify",
@ -15,7 +15,10 @@
"atom": ">0.50.0" "atom": ">0.50.0"
}, },
"dependencies": { "dependencies": {
"config-chain": "^1.1.8",
"emissary": "^1.0.0",
"extend": "^1.2.1",
"js-beautify": "~1.4.2", "js-beautify": "~1.4.2",
"emissary": "^1.0.0" "nopt": "^2.2.1"
} }
} }