Skip to content
This repository has been archived by the owner on Feb 22, 2018. It is now read-only.

Commit

Permalink
fix(cookies): Make sure Cookies is injectable.
Browse files Browse the repository at this point in the history
Also clean-up a few warnings and remove an unnecessary workaround
in the todo example.

Closes #856
  • Loading branch information
kasperl authored and mhevery committed Apr 4, 2014
1 parent 0281c06 commit 8952cbd
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 4 deletions.
3 changes: 0 additions & 3 deletions example/web/todo.dart
Original file line number Diff line number Diff line change
Expand Up @@ -72,9 +72,6 @@ class TodoController {
serverController.init(this);
}

// workaround for https://github.com/angular/angular.dart/issues/37
dynamic operator [](String key) => key == 'newItem' ? newItem : null;

void add() {
if (newItem.isEmpty) return;

Expand Down
1 change: 1 addition & 0 deletions lib/core_dom/cookies.dart
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,7 @@ class BrowserCookies {
/**
* Cookies service
*/
@NgInjectableService()
class Cookies {
BrowserCookies _browserCookies;
Cookies(this._browserCookies);
Expand Down
2 changes: 1 addition & 1 deletion test/io/test_files/templates/main.dart
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
library test_files.main;

import 'package:angular/core/module_internal.dart';
import 'package:angular/core/annotation.dart';
import 'package:angular/tools/template_cache_annotation.dart';

part 'partial.dart';
Expand Down

0 comments on commit 8952cbd

Please sign in to comment.