r61086@onn: sartak | 2008-05-26 21:31:16 -0400

[p5sagit/Devel-REPL.git] / lib / Devel / REPL / Plugin / Completion.pm

diff --git a/lib/Devel/REPL/Plugin/Completion.pm b/lib/Devel/REPL/Plugin/Completion.pm
index ae4a8ec..8fbe2bc 100644 (file)
--- a/lib/Devel/REPL/Plugin/Completion.pm
+++ b/lib/Devel/REPL/Plugin/Completion.pm
@@ -5,17 +5,23 @@ use PPI;
 use namespace::clean -except => [ 'meta' ];
 
 has current_matches => (
-    is => 'rw',
-    isa => 'ArrayRef',
-    lazy => 1,
-    default => sub { [] },
+  is => 'rw',
+  isa => 'ArrayRef',
+  lazy => 1,
+  default => sub { [] },
 );
 
 has match_index => (
-    is => 'rw',
-    isa => 'Int',
-    lazy => 1,
-    default => sub { 0 },
+  is => 'rw',
+  isa => 'Int',
+  lazy => 1,
+  default => sub { 0 },
+);
+
+has no_term_class_warning => (
+  isa => "Bool",
+  is  => "rw",
+  default => 0,
 );
 
 sub BEFORE_PLUGIN {
@@ -29,15 +35,26 @@ sub BEFORE_PLUGIN {
   };
 }
 
+sub AFTER_PLUGIN {
+  my ($self) = @_;
+
+  warn "Term::ReadLine::Gnu is required for the Completion plugin to work"
+    unless $self->term->isa("Term::ReadLine::Gnu") and !$self->no_term_class_warning;
+}
+
 sub _completion {
   my ($self, $text, $line, $start, $end) = @_;
 
   # we're discarding everything after the cursor for completion purposes
+  # we can't just use $text because we want all the code before the cursor to
+  # matter, not just the current word
   substr($line, $end) = '';
 
   my $document = PPI::Document->new(\$line);
   return unless defined($document);
 
+  $document->prune('PPI::Token::Whitespace');
+
   my @matches = $self->complete($text, $document);
 
   # iterate through the completions
@@ -60,5 +77,16 @@ sub complete {
   return ();
 }
 
+# recursively find the last element
+sub last_ppi_element {
+  my ($self, $document, $type) = @_;
+  my $last = $document;
+  while ($last->can('last_element') && defined($last->last_element)) {
+    $last = $last->last_element;
+    return $last if $type && $last->isa($type);
+  }
+  return $last;
+}
+
 1;