Fixed bug in CellEntry::setCellLocation() regex for extracting the cell location only handled single digits but now supports any row/column

asimlqt · asimlqt · commit 3f21660d938c · 2014-06-18T22:57:20.000+01:00
diff --git a/src/Google/Spreadsheet/CellEntry.php b/src/Google/Spreadsheet/CellEntry.php
@@ -151,7 +151,7 @@ public function update($value)
     protected function setCellLocation()
     {
         $id = $this->xml->id->__toString();
-        preg_match('@/R(\d)C(\d)@', $id, $matches);
+        preg_match('@/R(\d+)C(\d+)@', $id, $matches);
 
         if(count($matches) !== 3) {
             throw new Exception('Filed to get the location of the cell');

Original file line number	Diff line number	Diff line change
`@@ -151,7 +151,7 @@ public function update($value)`
`151`	`151`	`protected function setCellLocation()`
`152`	`152`	`{`
`153`	`153`	`$id = $this->xml->id->__toString();`
`154`		`- preg_match('@/R(\d)C(\d)@', $id, $matches);`
	`154`	`+ preg_match('@/R(\d+)C(\d+)@', $id, $matches);`
`155`	`155`
`156`	`156`	`if(count($matches) !== 3) {`
`157`	`157`	`throw new Exception('Filed to get the location of the cell');`