Bug #99741 for Games-Dukedom: Can't modify non-lvalue

Wed Oct 22 18:35:26 2014 DANAJ [...] cpan.org - Ticket created

Subject:

Can't modify non-lvalue

Grain for food [14]: 13 Land to buy at 19 HL/HA [0]: Land to sell at 18 HL/HA [0]: 100 Can't modify non-lvalue subroutine call at blib/lib/Games/Dukedom.pm line 634, <> line 37. which is: $self->land += $land->trades;

Thu Nov 06 22:00:28 2014 SILLYMOOS [...] cpan.org - Correspondence added

On Wed Oct 22 18:35:26 2014, DANAJ wrote: Show quoted text

> Grain for food [14]: 13 > Land to buy at 19 HL/HA [0]: > Land to sell at 18 HL/HA [0]: 100 > Can't modify non-lvalue subroutine call at blib/lib/Games/Dukedom.pm > line 634, <> line 37. > > which is: > > $self->land += $land->trades;

Changing this line to be $self->{land} Fixes the issue.

Thu Nov 06 22:00:28 2014 The RT System itself - Status changed from 'new' to 'open'

Thu Nov 06 22:24:53 2014 SILLYMOOS [...] cpan.org - Correspondence added

On Thu Nov 06 22:00:28 2014, SILLYMOOS wrote: Show quoted text

> On Wed Oct 22 18:35:26 2014, DANAJ wrote:

> > Grain for food [14]: 13 > > Land to buy at 19 HL/HA [0]: > > Land to sell at 18 HL/HA [0]: 100 > > Can't modify non-lvalue subroutine call at blib/lib/Games/Dukedom.pm > > line 634, <> line 37. > > > > which is: > > > > $self->land += $land->trades;

> > Changing this line to be > > $self->{land} > > Fixes the issue. >

Another one: Can't modify non-lvalue subroutine call at lib/Games/Dukedom.pm line 1078 Changing line 1078 to: $self->{grain} += $grain->spoils; resolves I think. Attached is patched *pm file.

Subject:

Dukedom.pm

Message body is not shown because it is too large.

Thu Nov 06 22:28:27 2014 jim [...] nortx.com - Correspondence added 10 min

Thank you! I am curious why my version of perl didn't show that. Fixed and released in v0.1.2 along with other changes. You should see it soon.

Thu Nov 06 22:28:27 2014 jim [...] nortx.com - Status changed from 'open' to 'resolved'

Thu Nov 06 22:28:28 2014 jim [...] nortx.com - Taken

Thu Nov 06 23:15:37 2014 jim [...] nortx.com - Correspondence added

Subject:	RE: [rt.cpan.org #99741] Can't modify non-lvalue
Date:	Thu, 6 Nov 2014 20:15:06 -0800
To:	<bug-Games-Dukedom [...] rt.cpan.org>
From:	"Jim Bacon" <jim [...] nortx.com>

Fixed. Uploaded to CPAN as v0.1.3. What version of Perl are you using? I'm not seeing the error on mine (5.16) and would like to check for it if possible, probably with perlview. Jim Another one: Can't modify non-lvalue subroutine call at lib/Games/Dukedom.pm line 1078 Changing line 1078 to: $self->{grain} += $grain->spoils; resolves I think. Attached is patched *pm file.

Sat Nov 08 12:34:32 2014 SILLYMOOS [...] cpan.org - Correspondence added

On Thu Nov 06 23:15:37 2014, BOFTX wrote: Show quoted text

> Fixed. Uploaded to CPAN as v0.1.3. What version of Perl are you using?

Hi Jim, Sure! I'm using Perl v5.20.0 all the best, David