Feature #2795

Update, add tests to verify if the current TXM user can install an update

Ajouté par Matthieu Decorde il y a plus de 5 ans. Mis à jour il y a plus d'un an.

Statut:Closed Début:17/04/2020
Priorité:Normal Echéance:
Assigné à:- % réalisé:

100%

Catégorie:Update Temps passé: -
Version cible:TXM 0.8.1

Description

Diagnostic

On windows, the user needs to start TXM with administrator rights to be able to install updates

Specific

FR

implémenter la solution par ordre de préférence (en fonction de ce qui est possible) :

  • solution 1)
    • commande "vérifier les mises à jours" :
      • si des mises à jour sont disponibles
        • alors si l'utilisateur est sur Windows && l'utilisateur n'a pas lancé TXM en tant qu'administrateur
          • alors ouvrir une boite de dialogue modale d'avertissement qui dit qu'il faut lancer TXM en tant qu'administrateur pour pouvoir installer des mises à jour et abandonner la commande en cliquant sur l'unique bouton OK
            et abandonner la commande en cliquant sur l'unique bouton OK
          • sinon procéder aux mises à jour
        • sinon ouvrir une boite de dialogue modale d'avertissement qu'il n'y a pas de mise à jour disponible et abandonner la commande en cliquant sur l'unique bouton OK
  • solution 2)
    • commande "vérifier les mises à jours" :
      • si l'utilisateur est sur Windows && l'utilisateur n'a pas lancé TXM en tant qu'administrateur
        • alors ouvrir une boite de dialogue modale d'avertissement qui dit :
          • "Il est nécessaire de lancer TXM en tant qu'administrateur pour pouvoir vérifier et installer les mises à jour. Merci de quitter TXM, de le relancer en tant qu'administrateur puis de relancer cette commande, pour pouvoir vérifier et installer les mises à jour."
          • et abandonner la commande en cliquant sur l'unique bouton OK
        • sinon procéder à la vérification et aux mises à jour le cas échéant
  • solution 3)
    • commande "vérifier les mises à jours" : si l'utilisateur est sur Windows && l'utilisateur n'a pas accès en écriture à des endroits nécessaires alors ouvrir une boite de dialogue modale d'avertissement qui dit qu'il faut être administrateur et abandonner la commande sinon vérifier les mises à jour et les installer le cas échéant
  • solution 4)
    • commande "vérifier les mises à jours" : si l'utilisateur est sur Windows alors ouvrir une boite de dialogue d'avertissement qui dit qu'il faut être connecté en tant qu'administrateur pour pouvoir installer des mises à jour, si clic sur Continuer continuer la commande jusqu'au bout sinon abandonner la commande

Solution

Implément solution 1.

private static boolean isWindowsAdminCommandTest()
    {
        Preferences preferences = systemRoot();

        synchronized (System.err)
        {
            setErr(new PrintStream(new OutputStream() {
                @Override
                public void write(int b) throws IOException { }
            }));

            try
            {
                preferences.put("TxmAdminAccessTestProperty", "TxmAdminAccessTestValue"); // SecurityException on Windows
                preferences.remove("TxmAdminAccessTestProperty");
                preferences.flush(); // BackingStoreException on Linux
                return true;
            } catch (Exception exception)
            {
                return false;
            } finally
            {
                setErr(System.err);
            }
        }
    }

If the Windows user is not detected as administrator a message is shown in a modal dialog box to 1) tell he must be administrator to install updates . When the "OK" button is pressed, the update is aborted.

Message per lang:
FR : Il est nécessaire de lancer TXM en tant qu'administrateur pour pouvoir vérifier et installer les mises à jour.
Merci de quitter TXM, de le relancer en tant qu'administrateur puis de relancer cette commande, pour pouvoir vérifier et installer les mises à jour.

EN : It is necessary to launch TXM as administrator to be able to check and install updates.
Please quit TXM, re-launch it as administrator, then re-launch this command to be able to check and install updates.

Title per lang:
FR: Mise à jour interrompue
EN: Update interrupted

Historique

#1 Mis à jour par Matthieu Decorde il y a plus de 5 ans

  • Description mis à jour (diff)

#2 Mis à jour par Matthieu Decorde il y a plus de 5 ans

  • Description mis à jour (diff)

#3 Mis à jour par Serge Heiden il y a plus de 5 ans

  • Description mis à jour (diff)

#4 Mis à jour par Matthieu Decorde il y a plus de 5 ans

  • % réalisé changé de 0 à 80
  • Description mis à jour (diff)

#5 Mis à jour par Serge Heiden il y a plus de 5 ans

  • % réalisé changé de 80 à 70
  • Description mis à jour (diff)

ajouté ouverture de boite de dialogue d'avertissement modale

#6 Mis à jour par Matthieu Decorde il y a plus de 5 ans

  • Description mis à jour (diff)
  • % réalisé changé de 70 à 40

#7 Mis à jour par Matthieu Decorde il y a plus de 5 ans

  • % réalisé changé de 40 à 60

implementation done, need testing on Windows

#8 Mis à jour par Matthieu Decorde il y a plus de 5 ans

  • % réalisé changé de 60 à 40
  • Description mis à jour (diff)

#9 Mis à jour par Matthieu Decorde il y a plus de 5 ans

  • Description mis à jour (diff)

#10 Mis à jour par Serge Heiden il y a plus de 5 ans

  • Description mis à jour (diff)

#11 Mis à jour par Matthieu Decorde il y a plus de 5 ans

  • Description mis à jour (diff)

#12 Mis à jour par Serge Heiden il y a plus de 5 ans

  • Description mis à jour (diff)

#13 Mis à jour par Matthieu Decorde il y a plus de 5 ans

  • % réalisé changé de 40 à 60
  • Description mis à jour (diff)

must fix the title

#14 Mis à jour par Matthieu Decorde il y a plus de 5 ans

  • % réalisé changé de 60 à 80

worked during the BETA tests

#15 Mis à jour par Sebastien Jacquot il y a plus d'un an

  • % réalisé changé de 80 à 100

#16 Mis à jour par Sebastien Jacquot il y a plus d'un an

  • Statut changé de New à Closed

Formats disponibles : Atom PDF