fixed insert many require react when the file has many require_tree

[towonzhou]

done

[facebook-github-bot]

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at [email protected]. Thanks!

[facebook-github-bot]

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

[towonzhou]

Does not need this?

[rmosolgo]

Since this doesn't have a test case, I want to read up inject_into_file before I merge it.

If you'd like to help me, you could add a test case which demonstrates the new behavior!

[rmosolgo]

I tried this out locally:

• First, I made an application.js which included only:

  //= require_tree ./application 
  //= require_tree ./something_else
  

• Using react-rails 1.2.0, I ran bundle exec rails g react:install.

• I observed the unwanted behavior. It added the react stuff to application.js twice:

  //= require react
  //= require react_ujs
  //= require components
  //= require_tree ./application
  //= require react
  //= require react_ujs
  //= require components
  //= require_tree ./something_else
  

• I reverted application.js to its original state.

• I pointed at this branch and bundle installed

  Using react-rails 1.4.1 (was 1.2.0) from git://github.com/towonzhou/react-rails.git (at master)
  

• I ran bundle exec rails g react:install

• I got the desired behavior. application.js contained:

  //= require react
  //= require react_ujs
  //= require components
  //= require_tree ./application
  //= require_tree ./something_else
  

[rmosolgo]

🎊 thank you for this fix!

[towonzhou]

@rmosolgo it's my pleasure.