[SalesForce] How to get https link using URLFOR() in Email Template

One approach might be to factor the bulk of your Email Template code (which of course is just VF markup anyway) into a Visualforce Component. Thus enabling you to share it by referencing this component between a Visualforce Page (to enable you to capture it on your attachments) and Email Templates (for when your emailing). For example.

<apex:component access="global">
  <apex:attribute name="targetObject" type="Test__c" description="The context of the outer page / email templates" access="global"/>
  <h1>Congratulations</h1>
  <p>This is your new Component that is shared between your Email Template and Visualforce Page</p>
  <p>This is the name of your record {!targetObject.Name}</p>
</apex:component>

Your email template would look like this...

<messaging:emailTemplate subject="Test" recipientType="User" relatedToType="Test__c">
    <messaging:plainTextEmailBody >
        <c:ShareComponent targetObject="{!relatedTo}"/>
    </messaging:plainTextEmailBody>
</messaging:emailTemplate>

Your VF page would look like this...

<apex:page standardController="Test__c">
    <c:ShareComponent targetObject="{!Test__c}"/>
</apex:page>

Hope this helps!

Note: Note that Visualforce Components when delivered in Managed Packages are none editable, unlike Email Templates. You can work around this though, by copy and pasting the component code into a new one in the subscriber code and changing the template to point to the new component.

This is a known issue with the pricebook2 object in Salesforce.

From the Apex Developers guide.

See here

http://www.salesforce.com/us/developer/docs/apexcode/index_Left.htm#StartTopic=Content/apex_security_sharing_rules.htm?SearchType=Stem

There is an idea you can vote on to get this feature here

https://success.salesforce.com/ideaView?id=08730000000ZE28AAG

As a workaround you can use the UserRecordAccess object in your Apex to only add the pricebook that the current user has access to, to the list of selectoptions. This is not ideal, but its a work around.

Note I did this in a text browser, so it hasnt been tested, but something like this should work for you

public SelectOption[] getPricebookList() {
    Pricebook2[] Records = [SELECT Name FROM Pricebook2 WHERE IsActive = true ORDER BY Name ASC];

    SelectOption[] Options = new SelectOption[]{};

    set<Id> pricebookIDs = new set<Id>();
    for(Pricebook2 R : Records){
        pricebookIDs.add(R.Id);
    }

    List<UserRecordAccess> listUserRecordAccess = [SELECT RecordId, HasReadAccess, HasEditAccess, 
       HasDeleteAccess FROM UserRecordAccess WHERE UserId=:UserInfo.getUserId() AND RecordId IN: pricebookIDs];

    map<Id,boolean> accessMap = new map<Id,boolean>();

    for(UserRecordAccess recAccess : listUserRecordAccess){
        accessMap.put(recAccess.RecordId, recAccess.HasReadAccess);
    }

    for(Pricebook2 R : Records){
        if(accessMap.get(R.Id)){
            Options.add(new SelectOption(R.Id, R.Name));
        }
    }

    return Options;
}