from Nathan Wallach -- comments on getUniqueName, problemUUID -- refactoring will be needed eventually about pg HOT 4 OPEN

I think the simplest fix for this is to just use the unique_id_stub that is defined in PGalias.pm. It incorporates the psvn, problemUUID, and courseID, as well as all of the things used in the $prob_name variable. I have tested it, and it works for the current cases, and should extend well to what Nathan wants. Just remove the $prob_name variable, and use $self->{unique_id_stub} instead.

Edit: Nevermind, I don't think that the unique_id_stub is as easily accessible there as I said. It doesn't work so easily at least.

Edit2: Wait, perhaps it does. I just didn't quit have the path right. Use $self->{PG_alias}{unique_id_stub}.

from pg.

I'll try to take a look. I don't think the unique_id_stub includes the problemSeed for example but I'm just talking from memory -- I'll have to go back and look when I have time to review the various sections of relevant code.

from pg.

The unique_id_stub does include everything that we have discussed. It definition is:

my $unique_id_seed = join("-",
			$self->{studentLogin},
			$self->{psvn},
			$self->{courseID},
			'set'.$self->{setNumber},
			'prob'.$self->{probNum},
			$self->{problemSeed},
			$self->{problemUUID},
);
my $unique_id_stub = create_uuid_as_string(UUID_V3, UUID_NS_URL, $unique_id_seed);
$self->{unique_id_stub} = $unique_id_stub;

from pg.

I somewhat suspect that just calling $main::PG->{PG_alias}->make_resource_object() already brings in the use of the $unique_id_seed (even without explicitly adding identity parameters to the "file name" portion) which is why the old code could just request the file with parameters
"image$imageNum","png".

That is why I wrote:

unless it is certain that these values will be taken into account elsewhere in the path/filename generation (probably based on the definition of $unique_id_stub which is made during the initialize() method in /lib/PGalias.pm ). I did not trace the code flow to see if that will happen.

Someone needs to check the code flow carefully.

If the ID parameters are pulled in automatically, when it suffices to use a generic name for $prob_name and not something which tries to pull in those values.

from pg.