Set the store attribute for newly created modules

[robin850]

Hi,

A regression has been introduced between RDoc 6.0.1 and 6.0.2 with ef958c2 where a new RDoc::NormalModule instance is created without a store associated to it.

This produces an error on generation through SDoc (see zzak/sdoc#124).

I don't know how to reproduce this through the test suite but here are some steps to reproduce it locally:

$ mkdir -p tmp/lib
$ cd tmp
$ echo "Foo::Bar" >> lib/bar.rb
$ echo "module Foo; end" >> lib/foo.rb

Then put the following content in a Rakefile:

require 'rdoc/task'
require 'sdoc'

RDoc::Task.new do |task|
  task.generator = 'sdoc'
end

Make sure to have sdoc installed or in your Gemfile and then run rake rdoc. This error isn't reproducible without using SDoc ; the culprit line is here which reaches this one and produces the error:

undefined method `rdoc' for nil:NilClass

This issue is pretty weird because parse order actually matters since the nested constant must be referenced before the top level constant definition (i.e. bar.rb must be parsed before foo.rb). Dunno whether the fix is correct or not but it works. 🤷‍♂️

Have a nice day !

[aycabta]

OK, I understand what this wants to do. This bug is my mistake. I guess that this Pull Request is correct. I'll add tests later.

[aycabta]

I gave up to add tests...I just merged. But I confirmed that this fix is correct.

[robin850]

Thanks a ton @aycabta !

[aycabta]

@robin850 I released RDoc 6.0.4. Please check it!

[robin850]

@aycabta : problem solved ; thanks for the quick release ! 🎉