[ 
https://issues.apache.org/jira/browse/RAVE-1104?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jmeas Pls updated RAVE-1104:
----------------------------

    Issue Type: Improvement  (was: Bug)

> Switch to alternate requireJS syntax
> ------------------------------------
>
>                 Key: RAVE-1104
>                 URL: https://issues.apache.org/jira/browse/RAVE-1104
>             Project: Rave
>          Issue Type: Improvement
>            Reporter: Jmeas Pls
>
> The problem:
> https://github.com/apache/rave/blob/angular/rave-portal-resources/src/main/webapp/static/script/common/resources/index.js#L20-L23
> Why is this a problem:
> This syntax is cumbersome, and from what I understand is only around for 
> legacy reasons. Using the CommonJS-style require syntax is much easier to 
> read and maintain, and is less error-prone. This is because you no longer 
> need to match up the values in the array with the method signature, which, as 
> the above file shows, quickly becomes unwieldy.
> The solution:
> The alternate syntax (docs here: http://requirejs.org/docs/whyamd.html#sugar)
> ```js
> define(function(require) {
>   var _ = require('underscore');
>   var Backbone = require('backbone');
> });
> ```



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to