webform_component_list() return value wrong when pagebreaks have digit names

I am pretty sure, but not 100% sure, that the page name (i.e. "label" in the UI) is intended only as a reference to the page. It can be any unique identifier. In this way, the code freely makes them unique by appending "_X" where X is a number.

In the situation you cite, would be possible to still have name conflicts. Suppose you have two identically-named pages "1". The first one would get renamed "1 " (with a space appended). The second one would also get the same name because there is no page name "1" yet in the array being built.

I suggest that the initial attempt for page names always have " " appended, as in:

    if ($pagebreak_groups && $component['type'] == 'pagebreak') {


      // Create a page name which is unique and cannot conflict with a cid.
      $page_name = $component['name'] . ' ';


      $copy = 1;
      while (in_array($page_name, $page_names)) {
        $page_name = $component['name'] . '_' . ++$copy;
      }
      $page_names[$page_num] = $page_name;
    }

What I don't know is which, if any, of the affected calls to webform_component_list with $pagebreak_groups == TRUE could be negatively affected. I would hope none. Conditionals seem okay.