juanje/debsfromrepos — comment on commit 38789d3865cf0bc4a2a5708199b7429461eae41a
File:  line 0
By ayosec:

The `get_translations_url` should not modify the `@lang` variable.

An [immutable object](http://en.wikipedia.org/wiki/Immutable_object) is a better approach.

The side effect of overriding the `@lang` is that you could leave the `DebsFromRepos` in a broken state. Something like this:

```ruby
o = DebsFromRepos.new("...", "", "", "es")
o.get_translations_url # This works. Use the @lang initialized in the constructor
o.get_translations_url("0") rescue nil # This does not work,
o.get_translations_url # This does not work anymore, since @lang is "0" now
```