ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ilya Borisov (JIRA)" <j...@apache.org>
Subject [jira] [Created] (IGNITE-9569) Web console: use $inject for DI instead of arrays
Date Thu, 13 Sep 2018 07:41:00 GMT
Ilya Borisov created IGNITE-9569:
------------------------------------

             Summary: Web console: use $inject for DI instead of arrays
                 Key: IGNITE-9569
                 URL: https://issues.apache.org/jira/browse/IGNITE-9569
             Project: Ignite
          Issue Type: Improvement
          Components: wizards
            Reporter: Ilya Borisov
            Assignee: Ilya Borisov


To do:
1. Remove provider registration by array spread, like this:
{code:java}
.service(...serviceArray){code}
 

Instead, use the canonical AngularJS approach:
{code:java}
.service('SeriveName', Service){code}
 

2. Do not use array Di syntax for exported symbols:
{code:java}
export ['IgniteVersion', 'Confirm', function directive (version, confirm){}]{code}
 

Instead, use $inject property:
{code:java}
export function directive(version, confirm) {}
directive.$inject = ['IgniteVersion', 'Confirm']{code}

Motivation:
The changes above will make older providers accessible to TypeScript, which in turn will allow
to increase type coverage.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message