Possible misnaming of answer blank subroutines about pg

[dpvc]

It seems to me that here, ans_rule is overruling the ans_rule from PGbasicmacros.pl

No, that's not the case. Since these are inside the package parserPopUp;here

pg/macros/parserPopUp.pl

Line 90 in 5e6c0a4

package parser::PopUp;

that makes the methods of the parserPopUp class, not global functions like main::answer_rule() that you refer to.

These are needed particular for things like PGML and MultiAnswer, which use $mo->ans_rule and $mo->named_ans_rule() to obtain answer rules for the MathObject $mo.

All of this is raising my suspicion that these instances were supposed to be the capital letter spelling, or something like that. Or do I misunderstand what is going on here?

I'm afraid it is the later. These are properly named. See

pg/macros/parserMultiAnswer.pl

Lines 445 to 461 in 5e6c0a4

	sub ans_rule {
	my $self = shift; my $size = shift || 20;
	my $data = $self->{data}[$self->{part}];
	my $name = $self->ANS_NAME($self->{part}++);
	if ($self->{singleResult} && $self->{part} == 1) {
	my $label = main::generate_aria_label($answerPrefix.$name."_0");
	return $data->named_ans_rule($name,$size,@_,aria_label=>$label);
	}
	if ($self->{singleResult} && $self->{part} > 1) {
	my $extension_ans_rule =
	$data->named_ans_rule_extension(
	$name,$size, answer_group_name => $self->{answerName},
	@_);
	# warn "extension rule created: $extension_ans_rule for ", ref($data);
	return $extension_ans_rule;
	} else {
	return $data->named_ans_rule($name,$size,@_);

and

pg/macros/PGML.pl

Lines 1010 to 1015 in 5e6c0a4

	my $method = ($item->{hasStar} ? "ans_array" : "ans_rule");
	if ($item->{name}) {
	unshift(@options,$item->{name});
	$method = "named_".$method;
	}
	$rule = $ans->$method(@options);

All MathObejcts get these methods from

pg/lib/Value/AnswerChecker.pm

Lines 363 to 371 in 5e6c0a4

	#
	# create answer rules of various types
	#
	sub ans_rule {shift; pgCall('ans_rule',@_)}
	sub named_ans_rule {shift; pgCall('NAMED_ANS_RULE',@_)}
	sub named_ans_rule_extension {shift; pgCall('NAMED_ANS_RULE_EXTENSION',@_)}
	sub ans_array {shift->ans_rule(@_)};
	sub named_ans_array {shift->named_ans_rule(@_)};
	sub named_ans_array_extension {shift->named_ans_rule_extension(@_)};

though some (like PopUps) need to override the basic method.

The upshot is, these are correct as they stand.

[Alex-Jordan]

OK, I get it. Thank you for the explanation. So these are essentially only there (right now) for the sake of parserMultiAnswer.pl. Sorry for the noise. I was trying to sort this all out while I try to mimic parserPopUp.pl for an attempt at a GoeGebra class.

[dpvc]

So these are essentially only there (right now) for the sake of parserMultiAnswer.pl

And also PGML.pl, and for creating answer arrays for matrices and other list-like MathObjects, but yes, it is for that.

[Alex-Jordan]

OK, I missed PGML using named_ans_rule given the way "named_ans_rule" isn't found in a search. I see it now.