Skip to content

Commit f31dcff

Browse files
committed
Remove app.del
1 parent 509ebb1 commit f31dcff

File tree

6 files changed

+5
-26
lines changed

6 files changed

+5
-26
lines changed

Diff for: History.md

+1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
===
33

44
* remove:
5+
- `app.del` - use `app.delete`
56
- `res.json(obj, status)` signature - use `res.json(status, obj)`
67
- `res.jsonp(obj, status)` signature - use `res.jsonp(status, obj)`
78
- `res.send(body, status)` signature - use `res.send(status, body)`

Diff for: lib/application.js

-5
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ var http = require('http');
1212
var compileETag = require('./utils').compileETag;
1313
var compileQueryParser = require('./utils').compileQueryParser;
1414
var compileTrust = require('./utils').compileTrust;
15-
var deprecate = require('depd')('express');
1615
var merge = require('utils-merge');
1716
var resolve = require('path').resolve;
1817
var slice = Array.prototype.slice;
@@ -444,10 +443,6 @@ app.all = function(path){
444443
return this;
445444
};
446445

447-
// del -> delete alias
448-
449-
app.del = deprecate.function(app.delete, 'app.del: Use app.delete instead');
450-
451446
/**
452447
* Render the given view `name` name with `options`
453448
* and a callback accepting an error and the

Diff for: test/app.del.js

-17
This file was deleted.

Diff for: test/app.listen.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@ describe('app.listen()', function(){
66
it('should wrap with an HTTP server', function(done){
77
var app = express();
88

9-
app.del('/tobi', function(req, res){
10-
res.end('deleted tobi!');
9+
app.get('/tobi', function(req, res){
10+
res.end('got tobi!');
1111
});
1212

1313
var server = app.listen(9999, function(){

Diff for: test/app.options.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ describe('OPTIONS', function(){
66
it('should default to the routes defined', function(done){
77
var app = express();
88

9-
app.del('/', function(){});
9+
app.post('/', function(){});
1010
app.get('/users', function(req, res){});
1111
app.put('/users', function(req, res){});
1212

Diff for: test/app.router.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ describe('app.router', function(){
3434
})
3535

3636
describe('methods', function(){
37-
methods.concat('del').forEach(function(method){
37+
methods.forEach(function(method){
3838
if (method === 'connect') return;
3939

4040
it('should include ' + method.toUpperCase(), function(done){

0 commit comments

Comments
 (0)