in reply to Programmatically Updating Code with PPI

I see a two problems 1) you're trying to insert at the wrong place (before assignment operator =), and 2) you're inserting the wrong thing (whitespace) 

#!/usr/bin/perl --

use strict;
use warnings;

use PPI;

my $perlcode = << "END_PERLCODE";
package My::L10N::en;

use base 'My::L10N';

our %Lexicon = (
    'Some [_1] text' =>
    'Some [_1] text',

);

END_PERLCODE

my $doc = PPI::Document->new(\$perlcode);
my $lexicon = $doc->find(sub { $_[1]->content eq '%Lexicon' });
$lexicon = $lexicon->[0]; # returns an array ref, but in my case, ther
+e's only ever one.

my $newmsg = q{'The [_1] is in the [_2]'};
my $newtext = sprintf '    %s =>\n    %s,', $newmsg, $newmsg;
my $insert_this = PPI::Document->new(\$newtext);

use PPI::Dumper;
print "\n tokens \n";
PPI::Dumper->new( $insert_this->tokens() )->print;
print "\n insert_this \n";
PPI::Dumper->new( $insert_this )->print;
print "\n lexicon \n";
PPI::Dumper->new( $lexicon)->print;
# insert $newtext into %Lexicon?
#~ $lexicon->insert_after($insert_this->tokens());
#~ $lexicon->insert_after($insert_this);
$lexicon->__insert_after($insert_this);

print "\n lexicon \n";
PPI::Dumper->new( $lexicon)->print;


print $doc->serialize();
__END__

[download]

 tokens 
PPI::Token::Whitespace      '    '

 insert_this 
PPI::Document
  PPI::Token::Whitespace      '    '
  PPI::Statement
    PPI::Token::Quote::Single      ''The [_1] is in the [_2]''
    PPI::Token::Whitespace      ' '
    PPI::Token::Operator      '=>'
    PPI::Token::Cast      '\'
    PPI::Token::Word      'n'
    PPI::Token::Whitespace      '    '
    PPI::Token::Quote::Single      ''The [_1] is in the [_2]''
    PPI::Token::Operator      ','

 lexicon 
PPI::Token::Symbol      '%Lexicon'

 lexicon 
PPI::Token::Symbol      '%Lexicon'
package My::L10N::en;

use base 'My::L10N';

our %Lexicon    'The [_1] is in the [_2]' =>\n    'The [_1] is in the 
+[_2]', = (
    'Some [_1] text' =>
    'Some [_1] text',

);

[download]

Replies are listed 'Best First'.
Re^2: Programmatically Updating Code with PPI
by Anonymous Monk on Jan 05, 2010 at 00:04 UTC
    Both insert_after and tokens/children seem to be broken methods (or unfinished), but I've managed to get it done using __insert_after 
    #!/usr/bin/perl --

use strict;
use warnings;

use PPI;

my $perlcode = << "END_PERLCODE";
package My::L10N::en;

use base 'My::L10N';

our %Lexicon = (
    'Some [_1] text' =>
    'Some [_1] text',

);

END_PERLCODE

my $doc = PPI::Document->new(\$perlcode);
my $lexicon = $doc->find_first(sub { $_[1]->content eq '%Lexicon' });

my $newmsg = q{'The [_1] is in the [_2]'};
my $newtext = sprintf qq'    %s =>\n    %s,\n', $newmsg, $newmsg;
my $insert_this = PPI::Document->new(\$newtext);


#~ use PPI::Dumper;
#~ print __LINE__,"\n"; PPI::Dumper->new( $lexicon->parent )->print;
#~ print __LINE__,"\n"; PPI::Dumper->new( $lexicon->parent->last_token
+ )->print;
#~ print __LINE__,"\n"; PPI::Dumper->new( $lexicon->parent->last_token
+->previous_token )->print;
#~ print __LINE__,"\n"; PPI::Dumper->new( $lexicon->parent->last_token
+->previous_token->previous_token )->print;
#~ print __LINE__,"\n"; PPI::Dumper->new( $insert_this->find_first('St
+atement') )->print;
#~ print __LINE__,"\n"; PPI::Dumper->new( $insert_this->find_first('St
+atement')->tokens )->print;
#~ print __LINE__,"\n"; PPI::Dumper->new( $insert_this->find_first('St
+atement')->children )->print;


$lexicon->parent->last_token->previous_token->previous_token
->__insert_after( $insert_this);

print $doc->serialize();


__END__
package My::L10N::en;

use base 'My::L10N';

our %Lexicon = (
    'Some [_1] text' =>
    'Some [_1] text',

    'The [_1] is in the [_2]' =>
    'The [_1] is in the [_2]',
);

    [download]
[reply]
[d/l]

In Section Seekers of Perl Wisdom