Convert remaining getAttribute() calls to ->: operator

No new problems found.

lib/xhp-bootstrap/alert.php

@@ -26,12 +26,12 @@ protected function compose(): :xhp {
       $child->addClass('alert-link');
     }
     $ret =
-      <div class={$this->getAttribute('class')}>
+      <div class={$this->:class}>
         {$this->getChildren()}
       </div>;
 
     $ret->addClass('alert');
-    $ret->addClass('alert-'.$this->getAttribute('use'));
+    $ret->addClass('alert-'.$this->:use);
 
     return $ret;
   }

lib/xhp-bootstrap/base.php

@@ -144,14 +144,14 @@ public function getID(): string {
   }
 
   public function requireUniqueID(): string {
-    if (!($id = $this->getAttribute('id'))) {
+    if (!($id = $this->:id)) {
       $this->setAttribute('id', $id = substr(md5(mt_rand(0, 100000)), 0, 10));
     }
     return $id;
   }
 
   public function addClass(string $class): this {
-    $this->setAttribute('class', trim($this->getAttribute('class').' '.$class));
+    $this->setAttribute('class', trim($this->:class.' '.$class));
     return $this;
   }
 

lib/xhp-bootstrap/button-group.php

@@ -28,10 +28,10 @@ enum {
   children ((:bootstrap:button|:bootstrap:button-group)+);
 
   protected function compose(): :xhp {
-    $class = $this->getAttribute('orientation') === 'horizontal'
+    $class = $this->:orientation === 'horizontal'
       ? 'btn-group' : 'btn-group-vertical';
     $ret = <div class={$class}>{$this->getChildren()}</div>;
-    switch ($this->getAttribute('size')) {
+    switch ($this->:size) {
       case 'large':
         $ret->addClass('btn-group-lg');
         break;
@@ -42,7 +42,7 @@ protected function compose(): :xhp {
         $ret->addClass('btn-group-xs');
         break;
     }
-    if ($this->getAttribute('justified')) {
+    if ($this->:justified) {
       $ret->addClass('btn-group-justified');
     }
 

lib/xhp-bootstrap/button.php

@@ -35,14 +35,14 @@ enum {
 
   protected function compose(): :xhp {
     $ret =
-      <a href={$this->getAttribute('href')}>
+      <a href={$this->:href}>
         {$this->getChildren()}
       </a>;
 
     $ret->addClass('btn');
-    $ret->addClass('btn-'.$this->getAttribute('use'));
+    $ret->addClass('btn-'.$this->:use);
 
-    switch ($this->getAttribute('size')) {
+    switch ($this->:size) {
       case 'large':
         $ret->addClass('btn-lg');
         break;
@@ -53,13 +53,13 @@ protected function compose(): :xhp {
         $ret->addClass('btn-xs');
         break;
     }
-    if ($this->getAttribute('block')) {
+    if ($this->:block) {
       $ret->addClass('btn-block');
     }
-    if ($this->getAttribute('active')) {
+    if ($this->:active) {
       $ret->addClass('active');
     }
-    if ($this->getAttribute('disabled')) {
+    if ($this->:disabled) {
       $ret->addClass('disabled');
     }
     return $ret;

lib/xhp-bootstrap/container.php

@@ -20,7 +20,7 @@ protected function compose(): :xhp {
     }
     $this->addClass("container");
     return
-      <div class={$this->getAttribute('class')}>
+      <div class={$this->:class}>
         {$this->getChildren()}
       </div>;
   }

lib/xhp-bootstrap/dropdown/item.php

@@ -25,7 +25,7 @@ protected function compose(): :xhp {
       </a>;
 
     $ret = <li role="presentation">{$link}</li>;
-    if ($this->getAttribute('disabled')) {
+    if ($this->:disabled) {
       $ret->addClass('disabled');
     }
     return $ret;

lib/xhp-bootstrap/glyphicon.php

@@ -218,7 +218,7 @@ enum {
 
   protected function compose(): :xhp {
     $this->addClass('glyphicon');
-    $this->addClass('glyphicon-'.$this->getAttribute('icon'));
+    $this->addClass('glyphicon-'.$this->:icon);
 
     return <span/>;
   }

lib/xhp-bootstrap/jumbotron.php

@@ -16,7 +16,7 @@ final class :bootstrap:jumbotron extends :bootstrap:base {
     string title;
 
   protected function compose(): :xhp {
-    $title = $this->getAttribute('title');
+    $title = $this->:title;
     if ($title) {
       $title = <h1>{$title}</h1>;
     }

lib/xhp-bootstrap/label.php

@@ -24,11 +24,11 @@ enum {
 
   protected function compose(): :xhp {
     $ret =
-      <span class={$this->getAttribute('class')}>
+      <span class={$this->:class}>
         {$this->getChildren()}
       </span>;
     $ret->addClass('label');
-    $ret->addClass('label-'.$this->getAttribute('use'));
+    $ret->addClass('label-'.$this->:use);
     return $ret;
   }
 

lib/xhp-bootstrap/list-group/list-group-item.php

@@ -25,11 +25,11 @@ enum {
 
   protected function compose(): :xhp {
     $ret = <a>{$this->getChildren()}</a>;
-    if ($this->getAttribute('active')) {
+    if ($this->:active) {
       $ret->addClass('active');
     }
     $ret->addClass('list-group-item');
-    switch ($this->getAttribute('use')) {
+    switch ($this->:use) {
       case 'success':
         $ret->addClass('list-group-item-success');
         break;

lib/xhp-bootstrap/navigation/brand.php

@@ -18,7 +18,7 @@ class :bootstrap:navbar:brand extends :bootstrap:base {
 
   protected function compose(): :xhp {
     return
-      <a href={$this->getAttribute('href')} class="navbar-brand">
+      <a href={$this->:href} class="navbar-brand">
         {$this->getChildren()}
       </a>;
   }

lib/xhp-bootstrap/navigation/nav.php

@@ -31,9 +31,9 @@ protected function compose(): :xhp {
         {$this->getChildren()}
       </ul>;
 
-    $ret->addClass('nav-'.$this->getAttribute('navstyle'));
+    $ret->addClass('nav-'.$this->:navstyle);
 
-    $layout_class = $this->getAttribute('layout');
+    $layout_class = $this->:layout;
     if ($layout_class != 'base') {
       $ret->addClass('nav-'.$layout_class);
     }

lib/xhp-bootstrap/navigation/navbar.php

@@ -27,8 +27,8 @@ enum {
   );
 
   protected function compose(): :xhp {
-    $theme = $this->getAttribute('theme');
-    $position = $this->getAttribute('position');
+    $theme = $this->:theme;
+    $position = $this->:position;
 
     $header = $this->getChildren('bootstrap:navbar:brand');
     if ($header) {

lib/xhp-bootstrap/navigation/navigation_link.php

@@ -23,8 +23,8 @@ class :bootstrap:navigation:link extends :bootstrap:base {
 
   protected function compose(): :xhp {
     return
-      <li class={$this->getAttribute('active') ? 'active' : null}>
-        <a href={$this->getAttribute('href')}>
+      <li class={$this->:active ? 'active' : null}>
+        <a href={$this->:href}>
           {$this->getChildren()}
         </a>
       </li>;

lib/xhp-bootstrap/page-header.php

@@ -17,7 +17,7 @@ class :bootstrap:page-header extends :bootstrap:base {
     string subtext;
 
   protected function compose(): :xhp {
-    $subtext = $this->getAttribute('subtext');
+    $subtext = $this->:subtext;
     if ($subtext !== null) {
       $subtext =
         <x:frag>
@@ -33,7 +33,7 @@ protected function compose(): :xhp {
 
     return
       <div class="page-header">
-        <h1>{$this->getAttribute('title')} {$subtext}</h1>
+        <h1>{$this->:title} {$subtext}</h1>
         {$lead}
       </div>;
   }

lib/xhp-bootstrap/panels/panel.php

@@ -30,12 +30,12 @@ enum {
 
   protected function compose(): :xhp {
     $ret =
-      <div class={$this->getAttribute('class')}>
+      <div class={$this->:class}>
         {$this->getChildren()}
       </div>;
 
     $ret->addClass('panel');
-    $ret->addClass('panel-'.$this->getAttribute('use'));
+    $ret->addClass('panel-'.$this->:use);
     return $ret;
   }
 

lib/xhp-bootstrap/progress-bar.php

@@ -30,9 +30,9 @@ enum {
     int max = 100;
 
   protected function compose(): :xhp {
-    $val = $this->getAttribute('value');
-    $min = $this->getAttribute('min');
-    $max = $this->getAttribute('max');
+    $val = $this->:value;
+    $min = $this->:min;
+    $max = $this->:max;
     if (($min >= $max) || ($val >= $max)) {
       $width = 100;
     } else if ($val < $max) {
@@ -42,16 +42,16 @@ protected function compose(): :xhp {
     }
     $style = "width: {$width}%";
     $ret =
-      <div class={$this->getAttribute('class')} style={$style}
+      <div class={$this->:class} style={$style}
            role="progressbar"
            aria-valuenow={$val}
            aria-valuemin={$min}
            aria-valuemax={$max}>
         {$this->getChildren()}
       </div>;
     $ret->addClass('progress-bar');
-    $ret->addClass('progress-bar-'.$this->getAttribute('use'));
-    switch ($this->getAttribute('stripes')) {
+    $ret->addClass('progress-bar-'.$this->:use);
+    switch ($this->:stripes) {
       case 'active':
         $ret->addClass('active');
         // FALLTHROUGH

lib/xhp-bootstrap/table.php

@@ -22,20 +22,20 @@ final class :bootstrap:table extends :bootstrap:base {
   protected function compose(): :xhp {
     $table = <table>{$this->getChildren()}</table>;
     $table->addClass('table');
-    if ($this->getAttribute('striped')) {
+    if ($this->:striped) {
       $table->addClass('table-striped');
     }
-    if ($this->getAttribute('border')) {
+    if ($this->:border) {
       $table->addClass('table-bordered');
     }
-    if ($this->getAttribute('hover-rows')) {
+    if ($this->:hover-rows) {
       $table->addClass('table-hover');
     }
-    if ($this->getAttribute('condensed')) {
+    if ($this->:condensed) {
       $table->addClass('table-condensed');
     }
 
-    if ($this->getAttribute('responsive')) {
+    if ($this->:responsive) {
       return <div class="table-responsive">{$table}</div>;
     }
     return $table;