0. Enable completion tracking at site level and for a test course.
1. On the test course, create a wiki and a survey. Set both to 'Mark complete automatically when conditions are met' and tick the 'Require view' condition.
2. Turn editing off and look at the course. Observe that the two activities are marked as not completed (blank icon with no tickbox).
3. Click into each activity in turn. In the case of the wiki, create the first page and put some content in and save it. (There needs to be a page in order for it to be counted as 'viewed'.) You don't need to do anything in the survey.
4. Look at the course again.

EXPECTED RESULT:
Both activities should now display a tick mark alongside, indicating that they have been completed because you viewed them.

0. Enable completion tracking at site level and for a test course.
1. On the test course, create a wiki and a survey. Set both to 'Mark complete automatically when conditions are met' and tick the 'Require view' condition.
2. Turn editing off and look at the course. Observe that the two activities are marked as not completed (blank icon with no tickbox).
3. Click into each activity in turn. In the case of the wiki, create the first page and put some content in and save it. (There needs to be a page in order for it to be counted as 'viewed'.) You don't need to do anything in the survey.
4. Look at the course again.
EXPECTED RESULT:
Both activities should now display a tick mark alongside, indicating that they have been completed because you viewed them.

the fix seemed perfect on github, 1 simple commit (47b6ced1) adding the viewed thing to survey and wiki. But if I merge your branch against integration I get 3 extra commits, by Charles Fulton, related to MDL-27242 (work in progress afaik, perhaps you messed your master?).

So this to ask for confirmation about simply cherry-picking 47b6ced1 (and backporting it) being correct, leaving Charles' stuff 100% out. Sounds ok? I'll be happy to do so after confirmation, TIA!

However I have deleted my master branch, recreated from upstream, and rebased so it should work the merge way too now if you want to do that. (Now fiddling with other commits in integration too, sigh...)

Sam Marshall
added a comment - 10/Aug/11 6:00 PM hi Eloy, argh! Yes the cherry-pick of the single commit is correct.
However I have deleted my master branch, recreated from upstream, and rebased so it should work the merge way too now if you want to do that. (Now fiddling with other commits in integration too, sigh...)